Skip to content

Add default “selected” button interactive states

Jeremy Elder requested to merge 940-add-default-selected-button-states into master

What does this MR do?

State Before After
Rest CleanShot_2020-08-21_at_08.55.53_2x CleanShot_2020-08-21_at_08.57.58_2x
Hover CleanShot_2020-08-21_at_08.55.53_2x CleanShot_2020-08-21_at_08.58.20_2x
Focus CleanShot_2020-08-21_at_08.55.53_2x CleanShot_2020-08-21_at_08.58.37_2x
Active CleanShot_2020-08-21_at_08.55.53_2x CleanShot_2020-08-21_at_08.58.57_2x
Active + Focus CleanShot_2020-08-21_at_08.56.43_2x CleanShot_2020-08-21_at_08.58.57_2x
CleanShot_2020-08-21_at_08.52.53 CleanShot_2020-08-21_at_08.51.32

Does this MR meet the acceptance criteria?

Conformity

  • Code review guidelines.
  • GitLab UI's contributing guidlines.
  • If it changes a Pajamas-compliant component's look & feel, the MR has been reviewed by a UX designer.
  • [-] If it changes GitLab UI's documentation guidelines, the MR has been reviewed by a Technical Writer.
  • [-] If the MR changes a component's API, integration MR(s) have been opened in the following projects to ensure that the @gitlab/ui package can be upgraded quickly after the changes are released:
  • Added the ~"component:*" label(s) if applicable.

Closes #940 (closed)

Edited by Jeremy Elder

Merge request reports