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

Issue 52 - add aria-hidden to taglines not shown #58

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

jamesros161
Copy link
Collaborator

ISSUE: Resolves #52

PROJECT: Crio 2.20.0

add aria-hidden to taglines not shown

Test Files

crio-2.20.0-issue-52.zip

NOTES TO TESTER

Please leave comments regarding issues found in testing directly on the issue linked above, not the Pull Request. This helps ease the process of reviewing the testing of multiple PRs in a given project from the project view.

Please remember to move this item from 'Needs Review' to either 'In Progress' or 'Awaiting Release' depending on the results of your testing

Testing Instructions

  1. Install the test zip files linked above.
  2. Ensure that the Tagline is not displayed.
  3. Inspect the header using Chrome DevTools, and ensure that the 'site-description' element has the aria-hidden=true attribute when it is not displayed.
  4. Enable the tagline to be displayed in the customizer.
  5. Inspect the header again and ensure that the aria-hidden=false attribute is set instead.

@nicolepaschen
Copy link
Member

looks good ty

@jamesros161 jamesros161 changed the base branch from master to branch-2.20.0 April 3, 2023 18:06
@jamesros161 jamesros161 merged commit aeb89bc into branch-2.20.0 Apr 3, 2023
@jamesros161 jamesros161 deleted the issue-52 branch April 3, 2023 18:06
@jamesros161 jamesros161 mentioned this pull request Apr 3, 2023
jamesros161 added a commit that referenced this pull request Apr 5, 2023
* update info dropdown for footer link colors (#55)

* add aria-hidden to taglines not shown (#58)

* add notice to logo panel (#59)

* Issue 50 - Improve Color Palettes by using color variables instead of SCSS compilation during runtime (#54)

* remove runtime compiling of SCSS color palettes

* add variable generation to customizer js

* append updated colors to head

* add css variables to saved palette theme_mod

* fix alternative parsing of palette values

* footer link color to work like other link controls

* add footer link selectors

* change footer link color control to new name

* set color pallette to postMessage

* ensure updated color palette values are available

* fix footer color updates

* update versions for alpha

* update side branch with master changes (#51)

* Create pull_request_template.md

* Branch 2.19.1 (#47)

* set rc release version

* Separate top & bottom blog page post margins

* Revert "Separate top & bottom blog page post margins"

This reverts commit 55585fd.

* Salon theme styles (#42)

* add salon theme styles

* update styles to fix MFW rows

* Fix Blog Category links not working - Issue 37 (#43)

* fix cats not showing if uncategorized

* fix category 404s

* Separate Blog Post List Post Margin controls - Issue 38 (#41)

* Split blog page post margins into two controls

* change title of lower margin control

* Fix Submenu Padding for first menu item - Issue 39 (#40)

* issue-786 version set

* fix left padding of first-item submenus

* update version for test-zip

* revert version to rc1

* add filters (#45)

* re-add letter-spacing (#19)

* update readme

* revert letter-spacing changes

* remove RC1 from version tag

* update versions and readme

* fix translucency issues

* fix SINCEVERSION

* fix incorrect '-raw' value in css

* add comments

* add comments

* remove commented out code

* add fallback for bgtfw_footer_links

* fix footer links. again.

* phpcs fixes

* fix raised buttons and overlays

* set versions to rc1

* fixed issues due to merge conflicts

* fix repeated changes not binding

* Rc1 fixes (#62)

* add colorx-background-alpha styles

* fix menu items font color not updating right

* update versions for stable

* update readme
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.

Hidden Tagline = Empty Heading Tag = Bad for Accessibility
3 participants