-
Notifications
You must be signed in to change notification settings - Fork 12
More obvious button states #464
base: develop
Are you sure you want to change the base?
Conversation
I think this should be visually approved, not just PR'ed. |
@elisechant Yup, I want Gary to be across these. |
can I suggest that on the 'disabled' button, we use the lightest text colour possible that is still 4.5:1 contrast, eg maybe #404040 or lighter |
Any chance of seeing a live working demo? |
@gazhands I’ll deploy one after lunch for you review. |
Thanks for the work so far everyone. I feel there's some more iteration on the visual design (of the buttons). Balancing all three states needs to be conceded (hover focus, and active) and getting these right, especially on touch screens. |
@gazhands Cool.
Could you elaborate? Does this mean we need to give this to the design clinic? I’m happy to work any design changes into the 2.x work, so knowing what the go is re. buttons soon would be great. (: |
Don't mind who picks this up but design clinic is a good start. Defo keen to have it for 2.0. |
*nod* We will likely look at buttons within the coming two weeks as we write 2.x, so I propose we bring it up tomorrow. |
Seeks to address #391
Could potentially expand from here but this is is a good start.
I used a negative offset so as to not affect the overall element size as it could affect surrounding elements if used say inline as part of an input group.
Definition of Done
npm test
)