Bug 329127 - Remove Tag menu entry is enabled if a picture has no Tags [patch]
Summary: Remove Tag menu entry is enabled if a picture has no Tags [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Usability-Menus (show other bugs)
Version: 4.0.0
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-22 15:31 UTC by André Marcelo Alvarenga
Modified: 2017-08-02 21:08 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.12.0


Attachments
contextmenu.patch (1.25 KB, patch)
2015-07-07 18:24 UTC, Maik Qualmann
Details

Note You need to log in before you can comment on or make changes to this bug.
Description André Marcelo Alvarenga 2013-12-22 15:31:29 UTC
If I have a picture with no Tags but with Labels, the 'Remove Tag' menu entry in Context Menu stayed enabled until I hover the mouse pointer over this menu entry.

Reproducible: Always

Steps to Reproduce:
1-Make sure that this picture has no Tag
2-Assign a Label
3-Right-click on this picture ('Remove Tag' menu entry is enabled)
4-Go to 'Remove Tag' menu entry


Expected Results:  
Don't enable menu entry if the picture has no Tags.
Comment 1 caulier.gilles 2013-12-22 21:03:13 UTC
Already implemented in 4.0.0-beta2

Gilles Caulier
Comment 2 André Marcelo Alvarenga 2014-06-12 02:07:40 UTC
(In reply to comment #1)
> Already implemented in 4.0.0-beta2
> 
> Gilles Caulier

I can still reproduce this bug on digikam 4.0
Comment 3 André Marcelo Alvarenga 2015-07-07 01:57:41 UTC
I can still reproduce this bug on digikam 4.11

Watch this screencast: https://youtu.be/gz0BdA24B9g
Comment 4 caulier.gilles 2015-07-07 16:23:40 UTC
As i can see on screencast, the menu entry is not disabled at menu showEvent(), but only when mouse is over menu option.

So, the disabling process must be done when menu is populated, just before to show it.

Gilles
Comment 5 Maik Qualmann 2015-07-07 18:24:16 UTC
Created attachment 93516 [details]
contextmenu.patch

This patch checks if only color or pick label tags in the selected images are.

Maik
Comment 6 caulier.gilles 2015-07-07 18:33:41 UTC
Just tested. Patch work fine here...

Gilles
Comment 7 Maik Qualmann 2015-07-07 19:02:56 UTC
Git commit 2426e9f6cbd150455e743e54cbc204f97ae47101 by Maik Qualmann.
Committed on 07/07/2015 at 18:50.
Pushed by mqualmann into branch 'master'.

apply patch #93516 to disable "Remove Tag" context menu if only color or pick label tags are present
FIXED-IN: 4.12.0

M  +2    -1    NEWS
M  +16   -2    app/utils/contextmenuhelper.cpp

http://commits.kde.org/digikam/2426e9f6cbd150455e743e54cbc204f97ae47101
Comment 8 Maik Qualmann 2015-07-07 20:09:32 UTC
Git commit b5badd9b0500f539dda24f85b052c9d1242ba85d by Maik Qualmann.
Committed on 07/07/2015 at 20:04.
Pushed by mqualmann into branch 'master'.

check also for internal tags to disable context menu

M  +2    -1    app/utils/contextmenuhelper.cpp

http://commits.kde.org/digikam/b5badd9b0500f539dda24f85b052c9d1242ba85d
Comment 9 caulier.gilles 2015-07-07 21:01:02 UTC
Git commit dee6477cec8aef873798011bec5598ebda7befd1 by Gilles Caulier.
Committed on 07/07/2015 at 21:00.
Pushed by cgilles into branch 'frameworks'.

backport commit #2426e9f6cbd150455e743e54cbc204f97ae47101 from git/master to frameworks branch

M  +17   -3    app/utils/contextmenuhelper.cpp

http://commits.kde.org/digikam/dee6477cec8aef873798011bec5598ebda7befd1
Comment 10 caulier.gilles 2015-07-07 21:02:38 UTC
Git commit b56d3002085d689b9cb8e0bc707d67f3964fc2a0 by Gilles Caulier.
Committed on 07/07/2015 at 21:02.
Pushed by cgilles into branch 'frameworks'.

backport commit #b5badd9b0500f539dda24f85b052c9d1242ba85d from git/master to frameworks branch

M  +2    -1    app/utils/contextmenuhelper.cpp

http://commits.kde.org/digikam/b56d3002085d689b9cb8e0bc707d67f3964fc2a0