| Summary: | Night light location symbol on the map in "Sunset and Sunrise at manual location" is below the cursor | ||
|---|---|---|---|
| Product: | [Applications] systemsettings | Reporter: | athaswildwhisper55232.untainted604 |
| Component: | kcm_nightcolor | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | athaswildwhisper55232.untainted604, dougshaw77, isma.af, kwin-bugs-null, natalie_clarius, nate |
| Priority: | NOR | Keywords: | qt6 |
| Version First Reported In: | 5.93.0 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/de8b79917c6c1dbd56f14bb547c1876ef871061c | Version Fixed/Implemented In: | 6.0 |
| Sentry Crash Report: | |||
|
Description
athaswildwhisper55232.untainted604
2024-02-05 12:30:06 UTC
Can reproduce. I cannot reproduce this on latest master but Qt6.6. Might be an upstream regression/change on `mapFromItem()` or `sceneEvent()`? Additional info: As I click any point in the map at Night Color/Sunset and Sunrise at manual location, this pops up in systemsettings log: qt.qml.context: qrc:/kcm/kcm_nightcolor/LocationsFixedView.qml:173:25 Parameter "eventPoint" is not declared. Injection of parameters into signal handlers is deprecated. Use JavaScript functions with formal parameters instead. FWIW I'm able to reproduce it with master on Qt 6.6.0. I use 225% scaling and 36px Breeze cursors, but I changed these to 200% and 24px (respectively) and there was no change. Same at 100% scale too. (In reply to Nate Graham from comment #4) > FWIW I'm able to reproduce it with master on Qt 6.6.0. I use 225% scaling > and 36px Breeze cursors, but I changed these to 200% and 24px (respectively) > and there was no change. Same at 100% scale too. I ran "systemsettings kcm_nightcolor" on Konsole terminal As I click on any location on the map in "Night color / sunset and sunrise at manual location", I see this message on my terminal: qt.qml.context: qrc:/kcm/kcm_nightcolor/LocationsFixedView.qml:173:25 Parameter "eventPoint" is not declared. Injection of parameters into signal handlers is deprecated. Use JavaScript functions with formal parameters instead. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3874 Git commit f8114bd5ae06c50e4b4821b5cbc752f032322392 by Ismael Asensio. Committed on 08/02/2024 at 16:29. Pushed by iasensio into branch 'master'. kcms/nightcolor: Fix click position for manual location To get the click point in local coordinates of the image, including its scaling transform, we can use the `position` property of the event point directly. See https://doc.qt.io/qt-6/qml-qtquick-handlerpoint.html#position-prop FIXED-IN: 6.0 M +2 -2 kcms/nightcolor/ui/LocationsFixedView.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/f8114bd5ae06c50e4b4821b5cbc752f032322392 Git commit de8b79917c6c1dbd56f14bb547c1876ef871061c by Ismael Asensio. Committed on 08/02/2024 at 18:33. Pushed by iasensio into branch 'Plasma/6.0'. kcms/nightcolor: Fix click position for manual location To get the click point in local coordinates of the image, including its scaling transform, we can use the `position` property of the event point directly. See https://doc.qt.io/qt-6/qml-qtquick-handlerpoint.html#position-prop FIXED-IN: 6.0 (cherry picked from commit f8114bd5ae06c50e4b4821b5cbc752f032322392) M +2 -2 kcms/nightcolor/ui/LocationsFixedView.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/de8b79917c6c1dbd56f14bb547c1876ef871061c |