[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 Eoin O'Neill changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/d746ef259 |hics/krita/commit/2e80b63dd |027927352fc4765b61ce3799b93 |f8d2e800cab45891c204ec4fac5 |3f4c|f552 --- Comment #8 from Eoin O'Neill --- Git commit 2e80b63ddf8d2e800cab45891c204ec4fac5f552 by Eoin O'Neill. Committed on 26/10/2021 at 20:38. Pushed by eoinoneill into branch 'krita/5.0'. Refine tablet right click popup palette behavior. Should be consistent now between AppImage and non-appimage build environments (different QT versions?). Originally, all tablet events were causing the time filter to get stuck, which would ensure that users had 1 right click event ignored. Also, event filter was inappropriately installed twice despite only needing it installed once due to refactor of popup palette design. This should streamline the right click event eating for the popup palette as a whole. If there's a regression on windows (I'll have to check tomorrow!) then I'll have to revert and try again. (cherry picked from commit d746ef259027927352fc4765b61ce3799b933f4c) M +4-3libs/ui/KisMouseClickEater.cpp M +0-1libs/ui/KisMouseClickEater.h M +1-2libs/ui/kis_popup_palette.cpp https://invent.kde.org/graphics/krita/commit/2e80b63ddf8d2e800cab45891c204ec4fac5f552 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 Eoin O'Neill changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/fe4204f9b |hics/krita/commit/d746ef259 |3c35631f9ae15278d96254e0d7b |027927352fc4765b61ce3799b93 |795c|3f4c Status|REOPENED|RESOLVED Resolution|--- |FIXED --- Comment #7 from Eoin O'Neill --- Git commit d746ef259027927352fc4765b61ce3799b933f4c by Eoin O'Neill. Committed on 20/10/2021 at 20:58. Pushed by eoinoneill into branch 'master'. Refine tablet right click popup palette behavior. Should be consistent now between AppImage and non-appimage build environments (different QT versions?). Originally, all tablet events were causing the time filter to get stuck, which would ensure that users had 1 right click event ignored. Also, event filter was inappropriately installed twice despite only needing it installed once due to refactor of popup palette design. This should streamline the right click event eating for the popup palette as a whole. If there's a regression on windows (I'll have to check tomorrow!) then I'll have to revert and try again. M +4-3libs/ui/KisMouseClickEater.cpp M +0-1libs/ui/KisMouseClickEater.h M +1-2libs/ui/kis_popup_palette.cpp https://invent.kde.org/graphics/krita/commit/d746ef259027927352fc4765b61ce3799b933f4c -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 Emmet O'Neill changed: What|Removed |Added Assignee|emmetoneill@gmail.com |eoinoneill1...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 --- Comment #6 from tomtomtomreportin...@gmail.com --- For some reason the issue only occurs in the appimages. It only takes one right-click to close the pop-up palette when using a locally compiled build of master. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 tomtomtomreportin...@gmail.com changed: What|Removed |Added Resolution|FIXED |--- Status|RESOLVED|REOPENED --- Comment #5 from tomtomtomreportin...@gmail.com --- I'm sorry, but while the issue of right-clicking never closing the triangle selector has been solved, the issue described by Yu-Hsuan Lai still occurs. The pop-up palette takes two right-clicks to dismiss across both selectors with a stylus. In Krita 4, it only took one right-click. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 Emmet O'Neill changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/b117b85f3 |hics/krita/commit/fe4204f9b |78f09027385f209a9333a43d353 |3c35631f9ae15278d96254e0d7b |8dcc|795c --- Comment #4 from Emmet O'Neill --- Git commit fe4204f9b3c35631f9ae15278d96254e0d7b795c by Emmet O'Neill. Committed on 22/09/2021 at 00:43. Pushed by emmetoneill into branch 'krita/5.0'. Popup Palette: Fixed a regression with right clicks. Some recent changes to the popup palette code caused a magic number value used by the timer within the right click eater event filter to no longer work correctly. This patch should fix that. (cherry picked from commit b117b85f378f09027385f209a9333a43d3538dcc) M +1-2libs/ui/KisMouseClickEater.cpp M +37 -0libs/ui/KisPopupWidgetInterface.h M +0-1libs/ui/KisView.cpp M +0-2libs/ui/canvas/kis_canvas2.cpp M +0-1libs/ui/input/KisPopupWidgetAction.cpp M +29 -21 libs/ui/input/kis_input_manager.cpp M +1-0libs/ui/input/kis_input_manager.h M +32 -24 libs/ui/kis_popup_palette.cpp M +6-2libs/ui/kis_popup_palette.h https://invent.kde.org/graphics/krita/commit/fe4204f9b3c35631f9ae15278d96254e0d7b795c -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 Emmet O'Neill changed: What|Removed |Added Status|CONFIRMED |RESOLVED Latest Commit||https://invent.kde.org/grap ||hics/krita/commit/b117b85f3 ||78f09027385f209a9333a43d353 ||8dcc Resolution|--- |FIXED --- Comment #3 from Emmet O'Neill --- Git commit b117b85f378f09027385f209a9333a43d3538dcc by Emmet O'Neill. Committed on 21/09/2021 at 22:11. Pushed by emmetoneill into branch 'master'. Popup Palette: Fixed a regression with right clicks. Some recent changes to the popup palette code caused a magic number value used by the timer within the right click eater event filter to no longer work correctly. This patch should fix that. M +1-2libs/ui/KisMouseClickEater.cpp M +37 -0libs/ui/KisPopupWidgetInterface.h M +0-1libs/ui/KisView.cpp M +0-2libs/ui/canvas/kis_canvas2.cpp M +0-1libs/ui/input/KisPopupWidgetAction.cpp M +29 -21 libs/ui/input/kis_input_manager.cpp M +1-0libs/ui/input/kis_input_manager.h M +32 -24 libs/ui/kis_popup_palette.cpp M +6-2libs/ui/kis_popup_palette.h https://invent.kde.org/graphics/krita/commit/b117b85f378f09027385f209a9333a43d3538dcc -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 --- Comment #2 from Emmet O'Neill --- Taking this one and also marking as a release blocker. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 Emmet O'Neill changed: What|Removed |Added Keywords||release_blocker CC||emmetoneill@gmail.com Assignee|krita-bugs-n...@kde.org |emmetoneill@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 441899] Pop-up palette doesn't dismiss itself at right click
https://bugs.kde.org/show_bug.cgi?id=441899 tomtomtomreportin...@gmail.com changed: What|Removed |Added Keywords||regression CC||tomtomtomreportingin@gmail. ||com Ever confirmed|0 |1 Status|REPORTED|CONFIRMED --- Comment #1 from tomtomtomreportin...@gmail.com --- Can confirm. There is differing behavior depending on the form of the color selector like in https://bugs.kde.org/show_bug.cgi?id=441154 . Behavior also differs between mouse and stylus. With triangle selector and a stylus, right-click inside the color selector area will never dismiss the pop-up palette, thus making it regressive from 4.4.7. With wide gamut selector and a stylus, it will take two clicks. With either selector and a mouse, it takes just one right-click. In 4.4.7, the pop-up palette takes two clicks to dismiss only when in the brush presets area, which I'm not sure is intended, but the selector area takes one right-click regardless of mouse or stylus. -- You are receiving this mail because: You are watching all bug changes.
