mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix toolbar icon size issue

Browse source 
Instead of messing with the margin (which didn't work, anyway), we need to
set the size of the icons. Apparently on Linux this was implicitly done,
but on Mac it didn't scale the icons and provided space for the largest
one (and we have a couple that are twice as big as the others).

What we really need are scalable icons that allow us to set the icon size
relative to the font size. But for now this solves the ugliness on Mac.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2015-01-04 07:15:23 -08:00
parent 71f3a29819
commit a308d2b7b4
1 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
qt-ui/mainwindow.cpp
View file

@ -127,9 +127,7 @@ MainWindow::MainWindow() : QMainWindow(),
Q_FOREACH (QAction *a, profileToolbarActions)
toolBar->addAction(a);
toolBar->setOrientation(Qt::Vertical);
QMargins margins(5, 5, -5, 5);
toolBar->setContentsMargins(margins);
toolBar->setIconSize(QSize(24,24));
// since I'm adding the toolBar by hand, because designer
// has no concept of "toolbar" for a non-mainwindow widget (...)
// I need to take the current item that's in the toolbar Position
@ -140,7 +138,7 @@ MainWindow::MainWindow() : QMainWindow(),
// and now for some layout hackery
// this gets us consistent margins everywhere and a much more balanced look
margins = QMargins(5, 5, 5, 5);
QMargins margins(5, 5, 5, 5);
QMargins zeroMargin(0, 0, 0, 0);
QList<QString> noMarginList;
noMarginList << "notesAndSocialNetworksLayout" <<