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

Remove forked Gutenberg nav block styles for sub menus #254

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

adamwoodnz
Copy link
Contributor

@adamwoodnz adamwoodnz commented Aug 26, 2022

See #246, some Gutenberg styles were forked to support navigation block sub menus in older browsers which don't support the :where selector.

The usage of this has now been removed in Gutenberg 13.9 so the forked styles can be removed.

Screenshots

Safari 13.1:

Screen Shot 2022-08-26 at 12 03 31 PM

Testing

  1. Build the global-header-footer styles to produce mu-plugins/blocks/global-header-footer/build/style.css
  2. Load this stylesheet in a local dev environment, eg. https://github.com/WordPress/wporg-main-2022 or sandbox
  3. Test the navigation sub menus across a range of browsers including those that don't support :where, eg. Safari 13.1, Falkon 3.1.0. Use mouse, keyboard, VoiceOver. Compare behaviour with wordpress.org

@adamwoodnz adamwoodnz marked this pull request as draft August 26, 2022 00:19
@adamwoodnz adamwoodnz self-assigned this Aug 26, 2022
@adamwoodnz adamwoodnz force-pushed the update/243-removed-forked-gb-nav-styles branch from dccaa41 to f3a7453 Compare August 26, 2022 00:25
@adamwoodnz adamwoodnz marked this pull request as ready for review August 26, 2022 03:57
Copy link
Contributor

@ryelle ryelle left a comment

Choose a reason for hiding this comment

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

Works in Chrome 87 👍🏻

@ryelle ryelle merged commit 0ec2219 into trunk Aug 29, 2022
@ryelle ryelle deleted the update/243-removed-forked-gb-nav-styles branch August 29, 2022 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants