[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 wolthera changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/cb7612c63 |hics/krita/commit/4211ee842 |2de3933ea7255f24813ec6da8f6 |16dafb078539eefebcbfa85521c |7e95|7fc7 --- Comment #10 from wolthera --- Git commit 4211ee84216dafb078539eefebcbfa85521c7fc7 by Wolthera van Hövell tot Westerflier. Committed on 14/06/2020 at 13:08. Pushed by woltherav into branch 'krita/4.3.0'. Use qt::window instead of qt::tool for keyboard events in the popups. The original code used the window hint instead of the tool hint, and that seems to be required to receive keyboard events. M +1-1 plugins/dockers/advancedcolorselector/kis_color_selector_base.cpp https://invent.kde.org/graphics/krita/commit/4211ee84216dafb078539eefebcbfa85521c7fc7 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 wolthera changed: What|Removed |Added Latest Commit||https://invent.kde.org/grap ||hics/krita/commit/6dc22af66 ||6658742b2106ee5c3ac5bf4d25e ||ed9e Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #8 from wolthera --- Git commit 6dc22af58742b2106ee5c3ac5bf4d25eed9e by Wolthera van Hövell tot Westerflier. Committed on 14/06/2020 at 13:03. Pushed by woltherav into branch 'master'. Use qt::window instead of qt::tool for keyboard events in the popups. The original code used the window hint instead of the tool hint, and that seems to be required to receive keyboard events. M +1-1 plugins/dockers/advancedcolorselector/kis_color_selector_base.cpp https://invent.kde.org/graphics/krita/commit/6dc22af58742b2106ee5c3ac5bf4d25eed9e -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 wolthera changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/6dc22af66 |hics/krita/commit/cb7612c63 |6658742b2106ee5c3ac5bf4d25e |2de3933ea7255f24813ec6da8f6 |ed9e|7e95 --- Comment #9 from wolthera --- Git commit cb7612c632de3933ea7255f24813ec6da8f67e95 by Wolthera van Hövell tot Westerflier. Committed on 14/06/2020 at 13:06. Pushed by woltherav into branch 'krita/4.3'. Use qt::window instead of qt::tool for keyboard events in the popups. The original code used the window hint instead of the tool hint, and that seems to be required to receive keyboard events. M +1-1 plugins/dockers/advancedcolorselector/kis_color_selector_base.cpp https://invent.kde.org/graphics/krita/commit/cb7612c632de3933ea7255f24813ec6da8f67e95 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 --- Comment #7 from Lynx3d --- No, commit 17e6e2841c245aa16eeb000f62f3e5717db7b657 was only about (not) signalling when painting with a new color, to build a color history. Shouldn't have any influence on the actual input handling. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 --- Comment #6 from Boudewijn Rempt --- Isn't this also fixed by commit 17e6e2841c245aa16eeb000f62f3e5717db7b657 Author: Mathias Wein Date: Tue Jun 9 20:04:08 2020 +0200 Remove old workaround for popup palette hiding ? -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 Lynx3d changed: What|Removed |Added CC||lynx.mw+...@gmail.com --- Comment #5 from Lynx3d --- I think this is a result of these commits: https://invent.kde.org/graphics/krita/-/commit/8621ed93b311bd8035390bb8063609b7b312dd24 https://invent.kde.org/graphics/krita/-/commit/cea9741dd7dbb37d4eaf812e7273d1c143c71217 >From what I remember, the first solution broke platforms other than macOS and I suggested to check if the Qt::Tool window flag works for all, but it didn't work for macOS either. Not sure why it was chosen for non-macOS in the end, but I wasn't aware that it changes behavior from the original flags either... -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 Boudewijn Rempt changed: What|Removed |Added Status|REPORTED|ASSIGNED CC||b...@valdyas.org Severity|wishlist|normal Ever confirmed|0 |1 Assignee|krita-bugs-n...@kde.org |griffinval...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 --- Comment #4 from Wojciech Trybus --- (In reply to vanyossi from comment #3) > Hello there, are you getting this issue on Kubuntu or in windows? on macos > color selector closes with any action, also ig the cursor leaves the color > selection. > > We implemented a change in cea9741dd7dbb for fixing bug 418668 that probably > introduced this bug on win and linux. It happens in both kubuntu and windows 10. Closing on mouse leaving the popup works fine - it's only not closing on keyboard inputs, that can perform actions (actions are being done, popup stays as it was), and it used to react to any input without performing its action. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 vanyossi changed: What|Removed |Added CC||ghe...@gmail.com --- Comment #3 from vanyossi --- Hello there, are you getting this issue on Kubuntu or in windows? on macos color selector closes with any action, also ig the cursor leaves the color selection. We implemented a change in cea9741dd7dbb for fixing bug 418668 that probably introduced this bug on win and linux. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 --- Comment #2 from Wojciech Trybus --- (In reply to wolthera from comment #1) > setting this to wishlist Of course, thank you, and sorry for the trouble. I made the bug report as Tiar asked me to on Krita-artists - according to Linx3d it was a change made by Ivan Yossi to fix another issue, and I thought it was probably unintended. Anyway it is not that severe, so I guess I will learn to live with it. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 419306] Advanced color selector not closing on keyboard inputs
https://bugs.kde.org/show_bug.cgi?id=419306 wolthera changed: What|Removed |Added Severity|normal |wishlist CC||griffinval...@gmail.com --- Comment #1 from wolthera --- setting this to wishlist -- You are receiving this mail because: You are watching all bug changes.
