Skip to content

Conversation

@WebsByTodd
Copy link
Collaborator

Nav highlight works pretty well, but it still has some edge cases that need to be worked out. Also need to fix the green highlight on the left-side for the group root pages.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 12, 2024

Deploying gitfromscratch with  Cloudflare Pages  Cloudflare Pages

Latest commit: fcf3ebf
Status: ✅  Deploy successful!
Preview URL: https://01f29e2f.gitfromscratch.pages.dev
Branch Preview URL: https://fix-nav-highlight.gitfromscratch.pages.dev

View logs

@nedtwigg
Copy link
Member

Suggestion:

  • drop the green line
  • add some saturation to the background highlight
  • image
  • image
  • image
image

Base automatically changed from fix/typography to main July 15, 2024 13:54
@WebsByTodd WebsByTodd changed the title WIP - Fix nav highlight Fix nav highlight Jul 15, 2024
@WebsByTodd
Copy link
Collaborator Author

This is ok, but still behaves less-than-ideal in certain situations. It's a tricky problem to solve. Happy to chat about it if you want.

@WebsByTodd WebsByTodd requested a review from nedtwigg July 15, 2024 17:51
@nedtwigg nedtwigg mentioned this pull request Jul 15, 2024
@WebsByTodd WebsByTodd merged commit ec6562f into main Jul 15, 2024
@WebsByTodd WebsByTodd deleted the fix/nav-highlight branch July 15, 2024 19:13
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.

2 participants