search

nurupo / ProjectTox-Qt-GUI

A cross-platform front end for ProjectTox Core library, written in C++11 with use of Qt5
GNU General Public License v3.0
341 stars 116 forks source link

Settings menu button #50

Closed Schlumpf closed 11 years ago

Schlumpf commented 11 years ago

Resubmit. See https://github.com/nurupo/ProjectTox-Qt-GUI/pull/42

nurupo commented 11 years ago

Is it just me, or the toolbar buttons seem to take quite a lot of space under the friend list?

before: before

after: after

nurupo commented 11 years ago

20x20 icon size looks better to me. What do you think?

after_20x20

Schlumpf commented 11 years ago

I agree, the icons were to large. I changed the size to 16x16 like mockup.

nurupo commented 11 years ago

Okay,16x16 it is then, although I thought it was just a little small.

after__

Schlumpf commented 11 years ago

16 is the mockup size. You can vary the size in the CSS, too. I wouldn't make them bigger, because of the mockup shows a large toolbar padding. And is will look better if we have the final icon set. For now, I think 16 is ok.