diff options
author | James Turner <james.turner@kdab.com> | 2012-06-26 14:43:20 +0100 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-06-27 00:46:35 +0200 |
commit | 9315497d2af6d6944792bbbffd61582c7061ef4c (patch) | |
tree | db149894c41bf3f0e3c692348fdba7577ccc48ff /src/widgets/widgets | |
parent | c0a0b3710302dc7711c4296c35df8ccf6b91f572 (diff) |
Convert QPA menus to use QIcon, and enable icons.
Icon support was disabled in QPA menus, now QIcon is in QtGui, convert the QPA interface, enable setting the icon in QMenu, and make the Cocoa implementation use it. (And fix a ref-counting leak if an icon is actually set)
Change-Id: Ica203bf6826b79d8beee58f39febc851b9633a66
Reviewed-by: Christoph Schleifenbaum <christoph.schleifenbaum@kdab.com>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
Reviewed-by: Bradley T. Hughes <bradley.hughes@nokia.com>
Diffstat (limited to 'src/widgets/widgets')
-rw-r--r-- | src/widgets/widgets/qmenu.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/widgets/widgets/qmenu.cpp b/src/widgets/widgets/qmenu.cpp index 4747d1c661..b201179661 100644 --- a/src/widgets/widgets/qmenu.cpp +++ b/src/widgets/widgets/qmenu.cpp @@ -2830,7 +2830,8 @@ void copyActionToPlatformItem(const QAction *action, QPlatformMenuItem* item) { item->setText(action->text()); item->setIsSeparator(action->isSeparator()); -// item->setIcon(action->icon()); + if (action->isIconVisibleInMenu()) + item->setIcon(action->icon()); item->setVisible(action->isVisible()); item->setShortcut(action->shortcut()); item->setChecked(action->isChecked()); |