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

Update header layout #271

Merged
merged 14 commits into from
Sep 13, 2023
Merged

Update header layout #271

merged 14 commits into from
Sep 13, 2023

Conversation

adamwoodnz
Copy link
Contributor

@adamwoodnz adamwoodnz commented Aug 29, 2023

Closes #268

Moves the breadcrumbs below the local nav bar, updates text and aligns the ToC.

Depends on WordPress/wporg-mu-plugins#439 and WordPress/wporg-parent-2021#103

Destination for Code Reference link TBC by design.

Screenshots

Home

Desktop Mobile Mobile nav open
localhost_8888_(Desktop) (7) localhost_8888_(Samsung Galaxy S20 Ultra) (9) localhost_8888_(Samsung Galaxy S20 Ultra) (8)

Code Reference

Desktop Mobile
localhost_8888_reference_functions_absint_(Desktop) (2) localhost_8888_reference_functions_absint_(Samsung Galaxy S20 Ultra) (2)

Coding Standards

Desktop Mobile
localhost_8888_coding-standards_inline-documentation-standards_php_(Desktop) (1) localhost_8888_coding-standards_inline-documentation-standards_php_(Samsung Galaxy S20 Ultra) (2)

Testing

  1. Run your plugins with responsive breadcrumbs
  2. Test the various page templates and check breadcrumbs and navigation layout and functionality

@adamwoodnz adamwoodnz changed the base branch from trunk to fix/243-code-reference-layout August 31, 2023 05:00
Base automatically changed from fix/243-code-reference-layout to trunk September 4, 2023 01:29
@adamwoodnz adamwoodnz force-pushed the update/268-header branch 3 times, most recently from aaf56db to dc5941c Compare September 4, 2023 01:39
@adamwoodnz adamwoodnz self-assigned this Sep 4, 2023
@adamwoodnz adamwoodnz added this to the MVP milestone Sep 4, 2023
@adamwoodnz adamwoodnz marked this pull request as ready for review September 4, 2023 03:25
@StevenDufresne
Copy link
Contributor

I tested this (without WordPress/wporg-parent-2021#103). LGTM

A couple non-blocking thoughts:

  • Should "Developer Resources" be a link?
  • Should we increase the font-weight for the active part (seems a bit subdued)

@adamwoodnz adamwoodnz marked this pull request as draft September 7, 2023 04:29
@adamwoodnz
Copy link
Contributor Author

I tested this (without WordPress/wporg-parent-2021#103). LGTM

A couple non-blocking thoughts:

  • Should "Developer Resources" be a link?
  • Should we increase the font-weight for the active part (seems a bit subdued)

Yeah this should use the local nav bar containing a site-title and navigation block like Main and Documentation. I'll update once those changes are shipped.

@adamwoodnz adamwoodnz force-pushed the update/268-header branch 2 times, most recently from d268d9f to 3d777fe Compare September 8, 2023 03:16
@adamwoodnz adamwoodnz marked this pull request as ready for review September 12, 2023 22:24
@adamwoodnz adamwoodnz merged commit 6b718d1 into trunk Sep 13, 2023
@adamwoodnz adamwoodnz deleted the update/268-header branch September 13, 2023 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Header update
2 participants