-
Notifications
You must be signed in to change notification settings - Fork 12
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
QSS Cleanups + FancyMenu for Clearlooks #118
base: master
Are you sure you want to change the base?
Conversation
I'm afraid we can't review this in those few remaining days now. |
2.1 release then? I didn't realize 2.0 was coming out this soon... |
Is there a way to just accept the dev team's lxqt-config.qss changes? My version of that file didn't do anything so there's no harm in overriding it. |
The file had just different contents between your version and master. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Spacer has no visible line|dots
anymoreit looks it was always buggy like that wallpaper.cfg has to be added again
themes/Clearlooks/lxqt-panel.qss
Outdated
border: 1px solid #5070a0; | ||
border-radius: 3px; | ||
background-color: #c9e1ff; | ||
#WingMenuPlugin QWidget { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as above
background: #edeceb; | ||
padding: 0px; | ||
margin: 0px; | ||
/* prevents unexpected borders/boxes on hover */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That doesn't work. Showdesktop, colorpicker, main|fancymenu, quicklaunch items have now borders on hover, while some others don't (volume, mount, brightness).
Okay, I think we're ready :) Removed the Wing Menu stuff, did some serious cleaning up in there to lower the verbosity, and tightened a few things up. This should be good to go. I'll do similar for Leech next. |
Not all mentioned issues are addressed. |
Cleaned up the code as requested (several months ago...sorry, I genuinely did not see it!) by @stefonarch - should be ready to merge for 2.0