extragear/graphics/digikam/digikam

SVN commit 968225 by aclemens:

It is more visible to use icons in the context menu, especially with our
quite big contextmenu, entries can get lost quite easy.
I added the tag icon for the "assign tag" / "remove tag" actions.

What do you think?
I think it helps a lot to identify those entries now.

Andi

CCMAIL:[hidden email]

 M  +2 -0      contextmenuhelper.cpp  


--- trunk/extragear/graphics/digikam/digikam/contextmenuhelper.cpp #968224:968225
@@ -322,6 +322,7 @@
 
     KMenu* assignTagsPopup = new TagsPopupMenu(ids, TagsPopupMenu::ASSIGN, d->parent);
     assignTagsPopup->menuAction()->setText(i18n("Assign Tag"));
+    assignTagsPopup->menuAction()->setIcon(SmallIcon("tag"));
     d->parent->addMenu(assignTagsPopup);
 
     connect(assignTagsPopup, SIGNAL(signalTagActivated(int)),
@@ -334,6 +335,7 @@
 
     KMenu* removeTagsPopup = new TagsPopupMenu(ids, TagsPopupMenu::REMOVE, d->parent);
     removeTagsPopup->menuAction()->setText(i18n("Remove Tag"));
+    removeTagsPopup->menuAction()->setIcon(SmallIcon("tag"));
     d->parent->addMenu(removeTagsPopup);
 
     // Performance: Only check for tags if there are <250 images selected
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

2009/5/15 Andi Clemens <[hidden email]>:
> SVN commit 968225 by aclemens:
>
> It is more visible to use icons in the context menu, especially with our
> quite big contextmenu, entries can get lost quite easy.
> I added the tag icon for the "assign tag" / "remove tag" actions.
>
> What do you think?
> I think it helps a lot to identify those entries now.

Fine for me...

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel