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

Mint-X: Add support for cinnamon 6.4 #487

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

fredcw
Copy link
Contributor

@fredcw fredcw commented Jan 6, 2025

Add support for cinnamon 6.4. Some things could probably use some tweaking: spacing, padding etc. but it all looks reasonable enough for now. I haven't implemented the dialog button destructive_action pseudo class (which I guess is optional anyway) and some old class names and perhaps unnecessary duplication could be removed. I couldn't test the Audio device selection dialog or network Auth dialog but they should look the same as the other dialogs.

Edit: Audio device selection dialog support added and tested. NetworkSecretDialog tested and ok.

Edit: Added dialog-button:destructive-action as red text. Changing button background color would mean creating a new set of button assets and the red button text looks ok anyway.

@clefebvre
Copy link
Member

Check the network password dialog. The visibility icon and the hint text in the entry are light on light.

@fredcw
Copy link
Contributor Author

fredcw commented Jan 10, 2025

Pretty sure that's fixed it but I can't test the network password dialog atm. It's fixed for keyring dialog and root auth dialog (open folder as root) so it should be the same.

@fredcw
Copy link
Contributor Author

fredcw commented Jan 11, 2025

Tested network password dialog. Fixed.

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.

2 participants