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.

Sign up for GitHub

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

Jump to bottom

Use chevron icon for local nav mobile menu #73

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Use chevron icon for local nav mobile menu #73

merged 1 commit into from
Oct 18, 2023

Conversation

adamwoodnz
Copy link
Contributor

@adamwoodnz adamwoodnz commented Oct 18, 2023 •
edited
Loading

Closes #72
Depends on WordPress/wporg-mu-plugins#480.

Screenshots

Page:State Before After
Home:Closed localhost_8888_(Samsung Galaxy S20 Ultra) (24) localhost_8888_(Samsung Galaxy S20 Ultra) (26)
Home:Open localhost_8888_(Samsung Galaxy S20 Ultra) (25) localhost_8888_(Samsung Galaxy S20 Ultra) (27)
Landing:Closed localhost_8888_overview_(Samsung Galaxy S20 Ultra) (1) localhost_8888_overview_(Samsung Galaxy S20 Ultra) (3)
Landing:Open localhost_8888_overview_(Samsung Galaxy S20 Ultra) (2) localhost_8888_overview_(Samsung Galaxy S20 Ultra) (4)
Article:Closed localhost_8888_article_block-directory_(Samsung Galaxy S20 Ultra) (3) localhost_8888_article_block-directory_(Samsung Galaxy S20 Ultra) (1)
Article:Open localhost_8888_article_block-directory_(Samsung Galaxy S20 Ultra) (4) localhost_8888_article_block-directory_(Samsung Galaxy S20 Ultra) (2)

How to test the changes in this Pull Request:

  1. Unsure your mu-plugins is on Add filter to replace local nav bar mobile menu icon wporg-mu-plugins#480
  2. Open home, a landing page and an article page
  3. Simulate mobile screen size
  4. Check that the icon in the mobile menu button is a chevron
  5. Open the menu
  6. Check the the chevron icon has been flipper vertically
  7. Close the menu

@adamwoodnz adamwoodnz added the [Component] Theme Templates, patterns, CSS label Oct 18, 2023
@adamwoodnz adamwoodnz self-assigned this Oct 18, 2023
@adamwoodnz adamwoodnz mentioned this pull request Oct 18, 2023
Merged
@adamwoodnz adamwoodnz marked this pull request as ready for review October 18, 2023 03:59
@adamwoodnz adamwoodnz requested review from StevenDufresne, renintw and a team October 18, 2023 04:00
@jasmussen
Copy link

Huh, I just commented on the other PR about lining up, and things look to be lining up here.

In any case, change looks good.

@adamwoodnz adamwoodnz merged commit 7d30caf into trunk Oct 18, 2023
1 check passed
@adamwoodnz adamwoodnz deleted the update/72-local-nav-use-chevron branch October 18, 2023 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StevenDufresne StevenDufresne Awaiting requested review from StevenDufresne

@renintw renintw Awaiting requested review from renintw

Assignees

@adamwoodnz adamwoodnz

Labels
[Component] Theme Templates, patterns, CSS Redesign
Projects
@adamwoodnz's WIP
Status: Done
WordPress.org
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use a chevron icon in the local nav mobile menu button
2 participants
@adamwoodnz @jasmussen