jajm / koha-staff-interface-redesign

Koha fork - only for staff interface redesign (bug 30952)
GNU General Public License v3.0
1 stars 1 forks source link

Save buttons for system preferences should be yellow #32

Open kfischer opened 2 years ago

kfischer commented 2 years ago

grafik

When editing an itemtype, the save button is yellow, but on other pages, like system preferences, the button is not really a button (no hover effect) and not yellow.

mrenvoize commented 2 years ago

I think we need a really clear style guide for when different button types should be used.

I agree here, I think the yellow is correct

kfischer commented 2 years ago

I think we are moving in the right direction here, but I didn't account for the fact that the buttons only activate when you changed a preference. This now causes a contrast issue on the 'unactivated' buttons.

Two options I think:

grafik

jajm commented 2 years ago

I'm not sure which patch did that, but it's a little bit better now: image

kfischer commented 2 years ago

It's a little better, but it looks like we are still failing the WCAG for contrast on those buttons.

jajm commented 2 years ago

By "those buttons", do you mean the yellow buttons (whatever their state is), or the yellow disabled buttons specifically ?

kfischer commented 2 years ago

We might be failing both (struggling a little with the tool), but the yellow diabled are worse.

mrenvoize commented 1 year ago

My accessibility checker considers the contrasts good here.. lets close this one.