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

[refs #160] Change the way triangle is generated in care cards #269

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

mcheung-nhs
Copy link
Collaborator

@mcheung-nhs mcheung-nhs commented Dec 1, 2018

Description

Using the clip path method for generating the care card triangles makes
it not show up in High Contrast Mode in Windows. This was picked up in a
DAC report. Despite looking at a gov.uk blog post on this
(https://accessibility.blog.gov.uk/2018/08/01/supporting-users-who-change-colours-on-gov-uk/)
it still didn't seem to work in High Contrast Mode, so it has been
changed to borders and transforms, inspired from
https://codepen.io/mpeace/pres/eBOVGe

Component checklist

  • SCSS
  • SCSS lint
  • HTML template
  • HTML validate & lint
  • Nunjucks macro
  • A standalone example
  • README/Documentation
  • Pseudocode tests
  • Visual tests
  • Tested against our testing policy (Resolution, Browser & Accessibility)
  • Print stylesheet considered
  • CHANGELOG

@chrimesdev chrimesdev force-pushed the accessibility/care-cards branch 2 times, most recently from 1dbd16a to dfebb8a Compare December 3, 2018 15:38
Using the clip path method for generating the care card triangles makes
it not show up in High Contrast Mode in Windows. This was picked up in a
DAC report. Despite looking at a gov.uk blog post on this
(https://accessibility.blog.gov.uk/2018/08/01/supporting-users-who-change-colours-on-gov-uk/)
it still didn't seem to work in High Contrast Mode, so it has been
changed to borders and transforms, inspired from
https://codepen.io/mpeace/pres/eBOVGe
@chrimesdev chrimesdev merged commit 0001ccc into master Dec 3, 2018
@chrimesdev chrimesdev deleted the accessibility/care-cards branch December 3, 2018 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants