Skip to content

Fix header sizes on account reset request page#5526

Merged
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/fix-heading-sizes
Oct 20, 2021
Merged

Fix header sizes on account reset request page#5526
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/fix-heading-sizes

Conversation

@mitchellhenke
Copy link
Contributor

@mitchellhenke mitchellhenke commented Oct 20, 2021

Follow up to #5522 to fix some accessibility issues that @anniehirshman-gsa caught

Old New
image image

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>
Copy link
Contributor

@anniehirshman-gsa anniehirshman-gsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thank you!

@mitchellhenke mitchellhenke merged commit 084931e into main Oct 20, 2021
@mitchellhenke mitchellhenke deleted the mitchellhenke/fix-heading-sizes branch October 20, 2021 16:06
mitchellhenke pushed a commit that referenced this pull request Oct 20, 2021
* fix header size on account reset request page

* Update app/views/account_reset/request/show.html.erb

Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>

Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>
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.

4 participants