diff options
author | Gerald Combs <gerald@wireshark.org> | 2014-09-22 09:59:02 -0700 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2014-09-23 02:42:14 +0000 |
commit | d3cd4c4b28d66d5d41a6c6a10bdf6f96187108a7 (patch) | |
tree | 9b54cf528685ec05b927ff9cd6a9028afcd39220 /ui/qt/wireshark_application.cpp | |
parent | be3246060e465d118afa8cc80a3015bdec8ffde6 (diff) |
Qt: Re-enable menu icons.
The GNOME HIG says "Where a corresponding toolbar icon exists, a menu
icon should mirror its design." The Windows and OS X HIGs say that you
should use them for common or familiar actions but otherwise avoid
clutter. I think icons in the "File" menu look like crap. Try to strike
a balance.
Use the 16x16 application icon for both the main web site and the wiki
since that's the favicon used on both sites.
Assume that "Reload" isn't used very often and remove it from the
toolbar.
Don't use a menu icon for the about box.
Other minor fixups.
Change-Id: I855211c218d266c2e9ed5acbe05a08750ab6d157
Reviewed-on: https://code.wireshark.org/review/4246
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'ui/qt/wireshark_application.cpp')
-rw-r--r-- | ui/qt/wireshark_application.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/ui/qt/wireshark_application.cpp b/ui/qt/wireshark_application.cpp index fb0991fbb6..89a70dec3f 100644 --- a/ui/qt/wireshark_application.cpp +++ b/ui/qt/wireshark_application.cpp @@ -593,8 +593,6 @@ WiresharkApplication::WiresharkApplication(int &argc, char **argv) : } #endif // Q_OS_WIN - setAttribute(Qt::AA_DontShowIconsInMenus, true); - QList<int> icon_sizes = QList<int>() << 16 << 24 << 32 << 48 << 64 << 128 << 256 << 512 << 1024; foreach (int icon_size, icon_sizes) { QString icon_path = QString(":/wsicon/wsicon%1.png").arg(icon_size); |