lxqt / lxqt-config

Tools to configure LXQt and the underlying operating system
https://lxqt.github.io
GNU Lesser General Public License v2.1
86 stars 60 forks source link

Added an option for toolbar icon size #1037

Closed tsujan closed 2 months ago

tsujan commented 2 months ago

NOTE 1: If the toolbar of an app like pcmanfm-qt gets misaligned after changing the size, just close and reopen its window.

NOTE 2: This option is for Qt apps of course. Codes could be added later for GTK apps too, but I don't believe it's a good idea.

tsujan commented 2 months ago

Only with Breeze it looks fine here

What do you mean by "only with Breeze"? It works especially with Fusion, which is our main target.

tsujan commented 2 months ago

Oh... :)

If you mean the sizes of combo-boxes or the alignment of texts, not only they have nothing to do with the PR, but also they're determined by the style (and its themes). For example, this is with Kvantum again, but with a different Kvantum theme:

kvantum

Strange that you hadn't noticed it before ;)

isf63 commented 2 months ago

It's probably my error, but I cannot get it to apply the icon size. I tried sudo make install and also sudo cp ./lxqt-config-appearance/lxqt-config-appearance /usr/bin/ - the combobox is there but nothing is applied.

tsujan commented 2 months ago

It's probably my error, but I cannot get it to apply the icon size.

You need https://github.com/lxqt/lxqt-qtplugin/pull/93 (that's merged now). After upgrading to it and restarting apps, please use Fusion for testing (see the tooltip).

isf63 commented 2 months ago

Ok, that made it work. I use Fusion typically, its aesthetic doesn't bother me. 22px is an interesting change from 24px. :)

tsujan commented 2 months ago

Thanks for testing!

I use Fusion typically, its aesthetic doesn't bother me.

It's simple and clean.

22px is an interesting change from 24px. :)

Also, try 16px; you might like it if you get used to it.

stefonarch commented 2 months ago

So there is no way to add an expandable spacer to cut down the length of the button?

tsujan commented 2 months ago

That's a "form layout", and it has always been so. Breeze centers it, Fusion doesn't, and Kvantum has an option for centering it.

This image shows a form layout of qpdfview drawn by Fusion (left) and by a Kvantum theme that centers form layouts (right):

form_layout