Skip to content

Reduce margins/padding for taskbar groups #141

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

isf63
Copy link
Contributor

@isf63 isf63 commented May 16, 2025

For most themes. Tested only with the Fusion Qt style. On Clearlooks:

Before:
before

After:
after

@isf63 isf63 requested a review from stefonarch May 16, 2025 15:04
@stefonarch
Copy link
Member

I never use them so I've no opinion about that. But needs testing with all widget styles IMO.

@isf63
Copy link
Contributor Author

isf63 commented May 16, 2025

I tested with 'Windows' style. It looks like QSS overrides all properties of the widget styles, at least it looks identical.

@tsujan
Copy link
Member

tsujan commented May 21, 2025

Sorry, but I didn't understand what was the problem that's supposed to be fixed by this change. Can't see an issue in the attached image either.

@tsujan
Copy link
Member

tsujan commented May 21, 2025

I trust @stefonarch's judgement on this. A review is already requested.

@isf63
Copy link
Contributor Author

isf63 commented May 27, 2025

Note: There was no bug here - no functionality was broken, this is just one stylistic choice over another.

I took better screenshots - the most important theme is Clearlooks as it is the default.

Ambiance before:
ambiance-before
Ambiance after:
ambiance-after
Clearlooks before:
clearlooks-before
Clearlooks after:
clearlooks-after

@isf63
Copy link
Contributor Author

isf63 commented Jun 14, 2025

@stefonarch

Qt styles only affect components using LXQt Themes for the un-themed parts, any existing theming overrides the widget style always. I tested Fusion/Windows and many Kvantum styles and there was no problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants