When creating a sub-tag of a tag, it seems the children inherits properties of his parents. It includes keyboard shortcut which is obviously bad and create a conflict which makes the shortcut useless.
Shortcut should be exclude from inheritance.
Steps to Reproduce:
1. have a tag with a shortcut defined
2. create a sub-tag
3. use the shortcut of the first tag
gui says there is a conflict: nothing done
write the corresponding tag
Git commit 1d0973cbdf8f57a2cf3329be1fcea0587f43df47 by Marcel Wiesweg.
Committed on 12/01/2013 at 22:05.
Pushed by mwiesweg into branch 'master'.
Do not keep shortcut of parent tag in New-Tag-Dialog: Would only create a conflict.
M +2 -1 NEWS
M +3 -2 digikam/tags/tageditdlg.cpp
With your, TagEdit dialog never show the shortcuts already assigned to a tag from DB.
It's very problematic, because user (me) think that tag has no shortcut assigned, and i try to re-assign it without success. But in background, yes shortcuts work perfectly...
You commit remove assignation of tag KS from DB through this line :
And there is no other code lines to show shortcut details by keySeqWidget. This one still always empty.
Q : What's the purpose of this commit ? If it fix something, it create a regression here...
Ok, then I edited the wrong location obviously.
The original bug was about a new tag being assigned the same shortcut as the parent, which is contradictory. A new tag should have no shortcut assigned by default.
worked on linux/3.4 but with the side-effect mentionned by Gilles
Test on 3.4 (lubuntu 13.10/kde 4.11.3 main install). pending on 3.5(macos 10.9/kde 4.11.4/macports, mostly w network share)(long collection scanning...)
The side effect of your previous patch is also reported in bug #338509
Git commit e0877e3205911cd42a792c7d071d638831f4cdec by Gilles Caulier.
Committed on 13/09/2014 at 09:42.
Pushed by cgilles into branch 'master'.
Add rules to not inherit of parent tags keyboard shortcut when creating ne tag.
Editing tags properties must show already assigned tag keyboard shortcut.
Related: bug 338509
M +3 -1 NEWS
M +13 -6 app/tags/tageditdlg.cpp
M +1 -1 app/tags/tageditdlg.h