Bug 348033 - Krita randomly decides mirroring view shortcut is ambiguous
Summary: Krita randomly decides mirroring view shortcut is ambiguous
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Usability (show other bugs)
Version: 2.9.2
Platform: Other Other
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-21 01:16 UTC by sqdqsddd
Modified: 2016-12-08 11:23 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description sqdqsddd 2015-05-21 01:16:34 UTC
Krita randomly decides that my shortcut for mirroring view (H) is ambiguous, even though it is not, after I close it in a certain way.

Reproducible: Always

Steps to Reproduce:
1.Open Krita and a document and set H as shortcut for mirroring view
2.Close document (NOT Krita yet)
3.Close Krita
4.Open Krita and load document via the splash screen menu or via file/open
5.Press H

Actual Results:  
Krita says the shortcut H is ambiguous

Expected Results:  
the image should be mirrored

-I have to go in the shortcut menu (just opening it is enough, I don't evn need to change anything) for the shortcut to work again. I have checked (even though I already know it since it works perfectly fine), nothing else uses H as shortcut (primary or alternate shortcut).
-if I close the document and open it again after having "fixed" the shortcut, it works fine. closing the document then Krita makes the bug occur again.
-Krita multiple document mode should be subwindows (didn't try with tabs)
Comment 1 Halla Rempt 2015-05-21 07:31:06 UTC
Hi,

Thanks for your report. It's not really random... The problem is that the color history short gets reset only in the session, but we don't save cleared shortcuts, so on restart, color history loads its H shortcut and mirror loads its H shortcut and we've got a conflict. I'm afraid that fixing this is going to take quite a lot of work since it involves code that we haven't written ourselves but comes from libraries :-(
Comment 2 Dmitry Kazakov 2016-12-08 10:48:38 UTC
Git commit fd407d5dd987b450738fccc3a3e2675c519aec33 by Dmitry Kazakov.
Committed on 08/12/2016 at 10:42.
Pushed by dkazakov into branch 'krita/3.1'.

Fix propertizing Color History and Common Colors actions

M  +2    -7    plugins/dockers/advancedcolorselector/kis_color_selector_ng_docker_widget.cpp

https://commits.kde.org/krita/fd407d5dd987b450738fccc3a3e2675c519aec33
Comment 3 Dmitry Kazakov 2016-12-08 11:23:06 UTC
Git commit 81453d3fd5461088090d4e0fb971ea579e9f06f5 by Dmitry Kazakov.
Committed on 08/12/2016 at 11:01.
Pushed by dkazakov into branch 'rempt/impex-refactoring'.

Fix propertizing Color History and Common Colors actions

M  +2    -7    plugins/dockers/advancedcolorselector/kis_color_selector_ng_docker_widget.cpp

https://commits.kde.org/krita/81453d3fd5461088090d4e0fb971ea579e9f06f5