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

a11y: add border to home-page buttons #2995

Merged
merged 7 commits into from
May 12, 2020
Merged

a11y: add border to home-page buttons #2995

merged 7 commits into from
May 12, 2020

Conversation

beyackle
Copy link
Contributor

Description

Adding a single-pixel border around the buttons on the home page makes them distinctive objects on the home page. We can't control what colors things turn in HC mode (at least not in a cross-browser way), so this seems like the next best solution.

Task Item

closes #2127

Screenshots

bot-buttons-with-border
bot-buttons-with-border-hc

@beyackle
Copy link
Contributor Author

I'd especially like @DesignPolice to weigh in here; is this a good solution?

@github-actions
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 71958ef on beyackle/hcButtons into 51d106e on master.

@DesignPolice
Copy link

Hey Ben,

I looked at this a few weeks ago, I guess the #'s got mixed up, anyway...

I don't have really strong opinions, I think it needs to be easy and not break a bunch of existing things.

I looked at the Dark Mode buttons in Fluent, they don't have a High Contrast version released yet. Here is what I did for the three options. We can discuss based on the amount of work it would require.
Dark Mode link:
https://www.figma.com/file/50J0ZSgM9QyV2qSYPff4SS/Fluent-Web-Dark-Mode-Archive?node-id=0%3A2741

Figma link to my version:
https://www.figma.com/file/Ux0vucOvaLL2xH22WpFgV65X/MarcB_Composer-UI?node-id=5584%3A0

Screen grab of mentioned Bostian Figma link

Screen Shot 2020-05-11 at 12 34 49 PM

@beyackle
Copy link
Contributor Author

Hey Ben,

I looked at this a few weeks ago, I guess the #'s got mixed up, anyway...

I don't have really strong opinions, I think it needs to be easy and not break a bunch of existing things.

I looked at the Dark Mode buttons in Fluent, they don't have a High Contrast version released yet. Here is what I did for the three options. We can discuss based on the amount of work it would require.
Dark Mode link:
https://www.figma.com/file/50J0ZSgM9QyV2qSYPff4SS/Fluent-Web-Dark-Mode-Archive?node-id=0%3A2741

Figma link to my version:
https://www.figma.com/file/Ux0vucOvaLL2xH22WpFgV65X/MarcB_Composer-UI?node-id=5584%3A0

Screen grab of mentioned Bostian Figma link

Screen Shot 2020-05-11 at 12 34 49 PM

I saw that, but the problem is that there's no supported standard for a browser to actually detect high-contrast mode and keep the colors in place. Edge has a special extension for it, but that's it; any solution using that extension would only ever work in Edge under Windows. Since we can't control what the colors do in high-contrast mode (or even detect if high-contrast mode is on or off), we need a solution involving shapes instead.

@DesignPolice
Copy link

DesignPolice commented May 11, 2020

Ah okay - no worries, well then if we can get some outlines, and some way to distinguish focus and things we can go with that. No worries this close to a launch. Thanks Ben

@beyackle
Copy link
Contributor Author

image
Here's how the focus indicator looks in high-contrast mode. This change is literally just a normal border around the element; there's nothing fancy going on, so focus indication will work as well as it usually does.

@DesignPolice
Copy link

alright well, if that is how it looks on the other buttons lets just go with that... I can see a visual pass of the different modes in my future... but if this will make our a11y peeps happy then I can live with it as well.

@beyackle
Copy link
Contributor Author

Note: those failing checks are a known security issue in lodash and nothing to do with this particular change.

@a-b-r-o-w-n a-b-r-o-w-n added the Approved to merge approved, waiting to be merged label May 11, 2020
@cwhitten cwhitten merged commit 51ea3db into master May 12, 2020
@cwhitten cwhitten deleted the beyackle/hcButtons branch May 12, 2020 14:46
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Andy Brown <asbrown002@gmail.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved to merge approved, waiting to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants