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

disable apply button, fix button ripples bug #579

Merged
merged 7 commits into from
Mar 30, 2023
Merged

Conversation

tameremad
Copy link
Contributor

@tameremad tameremad commented Mar 27, 2023

  • make the deploy button disabled when a version is already deployed.
  • use the button--unelevated class so that contained/filled buttons would have ripples too.
  • also use this class for the sidebar and the servicelane buttons
  • fix a bug with the add lock button with white space wrapping
  • fix bug with store returning empty release for undeployed versions
    before:
    image
    after:
    image
    fixes the ripple.
    Screencast from 29.03.2023 16:49:34.webm

SRX-HLVCYS

@sven-urbanski-freiheit-com
Copy link
Contributor

this is hard to review, please add before/after screenshots

Copy link
Contributor

Choose a reason for hiding this comment

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

please clean up the commit message before merging

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