Skip to content
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

add platform-specific default.qss #10393

Open
mixxxbot opened this issue Aug 23, 2022 · 3 comments
Open

add platform-specific default.qss #10393

mixxxbot opened this issue Aug 23, 2022 · 3 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: ronso0
Date: 2021-04-14T17:05:10Z
Status: New
Importance: Wishlist
Launchpad Issue: lp1923876
Tags: skin


default-mac.qss
default-linux.qss
default-windows.qss

Right now there's a lot of redundancy across skins in style sheets for most Mixxx menus like WTrackMenu and  #MainMenu menubar; and for complete theming styles for the Track Poperties window, tag fetcher and Preferences will be added, too.

To ease maintenance of all that let's move the common proven styles to a central place.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2021-04-14T23:48:53Z


that was easy, implemented here
https://github.com/ronso0/mixxx/tree/platform-default-styles

put on hold for now (2.3) because

  • for custom skins make the main menubar hard to use because any property set for #MainMenu would remove the OS palette, thus there would be no selection highlight and no checkmark icons
  • Mixxx skin's menu styles make use of '#LibraryContainer QMenu...' (shortcut to select QMenus in WLibrarySidebar , WLibrary, WSearchlineEdit, ...) and that doesn't pass the qsscheck pre-commit hook (yet) because #LibraryContainer is not set anywhere in c++ and is probably not used in custom skins, thus custom skins would have inconsistent, broken menus

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2021-06-25T16:22:53Z


Can this one be closed?

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2021-06-27T20:48:58Z


not really.
recently I introduced default-menu-styles-[windows, linux].qss, though those are not parsed by the skinloader like default.qss but just set in the Style node of skin.xml via src-win and src-linux.
for mac we still need custom stylesheets to work around some strange click/release behaviour (skin setting buttons in Deere and Tango).
That said, it would still be convenient to have the requested platform styles parsed without having to specify them in each skin.xml

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@ronso0 ronso0 added skins and removed skin labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants