bcgov / ocp-sso

BCGov Single Sign-On KeyCloak
http://oidc.gov.bc.ca/
Apache License 2.0
8 stars 8 forks source link

Update bcgov theme in compliance to WCAG 2.1 AA #50

Open ShellyXueHan opened 5 years ago

ShellyXueHan commented 4 years ago

@wenzowski any update on this?

wenzowski commented 4 years ago

@hamzajaved was looking at this and if I recall it got reprioritized. I would expect this to still be on our radar since we need to get the username/password authentication working in the bcgov theme at the same time.

wenzowski commented 4 years ago

@Maralsotoudehnia where did we land on reprioritizing this?

Maralsotoudehnia commented 4 years ago

@wenzowski @ShellyXueHan IIRC @hamzajaved mocked up some designs but then everyone got super busy and the work to implement changes turned out to be bigger than anticipated so it got deprioritized. Happy to pick this up again after we launch our app and the dust settles a bit. Maybe we can revive this collab in July/August? If so, it would be helpful to know what acceptance criteria Platform Services is looking for. I don't think it's effective to work without a shared understanding of what DoD would look like. Happy to chat.

wenzowski commented 4 years ago

@hamzajaved Is your design work ready to be posted publicly in this issue for peer review by #ux-guild?

ShellyXueHan commented 4 years ago

It's great to hear about this! And yea agree with Alec to get some eyes on it. Also if you need some design details, see here

hamzajaved commented 4 years ago

Hey @ShellyXueHan. Yes, I have worked on the BCGov theme for KeyCloak login based on the XD you provided. Some notable updates:

The designs can be reviewed here: https://xd.adobe.com/view/4d5292c8-b4ad-4a5d-7dec-d077dacd0f93-3b63/?fullscreen&hints=off

ShellyXueHan commented 4 years ago

it's looking great! Let me know when you are ready to start implement. I'll setup the sandbox KeyCloak instance for you again (it's scaled down to reduce resource consumption).

hamzajaved commented 4 years ago

Login Options.xd.zip

I have attached my working XD file. It's ready for review by #ux-guild. @dlevineBC, you will see that I've converted components, type, and colors into usable classes in the XD. Let me know if that is helpful.

dlevineBC commented 4 years ago

Looks great!

wenzowski commented 4 years ago

Where do we expect the “I’m an admin” links to go?

Love the “Get a BCeID” button. I recall we were informed there’s no way to set a return redirect on that signup process. Maybe that button should open a new window with some indication of the altered behaviour (eg new window icon).

wenzowski commented 4 years ago

Are the FAQs/IDP guidance intended to be configurable? Can we standardize them and bake approved copy directly into the theme?