diff options
author | Andy Shaw <andy.shaw@digia.com> | 2014-09-18 20:49:41 +0200 |
---|---|---|
committer | Richard Moe Gustavsen <richard.gustavsen@digia.com> | 2014-09-20 18:23:35 +0200 |
commit | b53f6fdd313f71f05b99093966736a8f7442ded8 (patch) | |
tree | 3c71f6f7b3b516595dac3ba20a9fb3b54ae676c2 /src/widgets/widgets/qmenu.cpp | |
parent | 4d8dd7f673d874607eb23c0dc36bb7c24324aa4f (diff) |
Respect the PM_SmallIconSize setting for icons in a menu on OS X
When a platform menu is used then it would hard code the icon size to
16x16. Instead of using the hard coded value then PM_SmallIconSize should
be used instead.
Change-Id: I27540ebc4397501e8f57686a118c28cd7167c0a1
Reviewed-by: Morten Johan Sørvig <morten.sorvig@digia.com>
Diffstat (limited to 'src/widgets/widgets/qmenu.cpp')
-rw-r--r-- | src/widgets/widgets/qmenu.cpp | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/widgets/widgets/qmenu.cpp b/src/widgets/widgets/qmenu.cpp index 846199ebe2..94cdf2e763 100644 --- a/src/widgets/widgets/qmenu.cpp +++ b/src/widgets/widgets/qmenu.cpp @@ -2978,8 +2978,17 @@ static void copyActionToPlatformItem(const QAction *action, QPlatformMenuItem* i { item->setText(action->text()); item->setIsSeparator(action->isSeparator()); - if (action->isIconVisibleInMenu()) + if (action->isIconVisibleInMenu()) { item->setIcon(action->icon()); + if (QWidget *w = action->parentWidget()) { + QStyleOption opt; + opt.init(w); + item->setIconSize(w->style()->pixelMetric(QStyle::PM_SmallIconSize, &opt, w)); + } else { + QStyleOption opt; + item->setIconSize(qApp->style()->pixelMetric(QStyle::PM_SmallIconSize, &opt, 0)); + } + } item->setVisible(action->isVisible()); item->setShortcut(action->shortcut()); item->setCheckable(action->isCheckable()); |