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

[55132] Disabled Primer button styles not displayed correctly #15601

Conversation

HDinger
Copy link
Contributor

@HDinger HDinger commented May 17, 2024

Use darker version for disabled primary button style and set border as well

https://community.openproject.org/projects/openproject/work_packages/55132/activity

Before

Bildschirmfoto 2024-05-17 um 13 10 23

After

Bildschirmfoto 2024-05-17 um 13 09 59

@jjabari-op
Copy link
Collaborator

@HDinger I've checked out this branch, updated all packages (npm and bundle), restarted the rails and angular server. The button is now completely invisible when disabled (Chrome and Firefox). The button is rendered but all white as the background. I'm using the default theme ("Light"). In "Ligh High Contrast" the button seems to be correctly styled however!

@HDinger HDinger force-pushed the bug/55132-disabled-primer-button-styles-not-displayed-correctly branch from b1d6702 to 8645918 Compare May 17, 2024 11:50
@HDinger
Copy link
Contributor Author

HDinger commented May 17, 2024

@jjabari-op Good catch, I forgot to add the defaults, for the case that no custom style is configured. 👍 Could you please try again?

Copy link
Collaborator

@jjabari-op jjabari-op left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks perfect now!

@HDinger HDinger merged commit 1deb14e into release/14.1 May 17, 2024
5 checks passed
@HDinger HDinger deleted the bug/55132-disabled-primer-button-styles-not-displayed-correctly branch May 17, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants