Skip to content
This repository was archived by the owner on Feb 15, 2022. It is now read-only.

Implement header navigation component #125

Open
4 of 6 tasks
ghost opened this issue Mar 11, 2021 · 7 comments
Open
4 of 6 tasks

Implement header navigation component #125

ghost opened this issue Mar 11, 2021 · 7 comments
Assignees

Comments

@ghost
Copy link

ghost commented Mar 11, 2021

  • Implement non-interactive menu for mobile and large screens
  • Implement a minimal interactive menu for large screens
  • Implement a minimal, interactive menu for mobile screens
  • Update to match current large screen and mobile screen Figma designs (could be a few hours or two days, depending on complexity of CSS)
  • Change large screen menu to use hidden instead of opacity to fix subtle issue with hidden menu capturing mouse events
  • Change large screen menu to use z-order to ensure dropdown menus always appear above other content
@ghost
Copy link
Author

ghost commented Mar 17, 2021

  • Implement a simple, interactive menu for large screens

Done in #160

@ghost ghost changed the title Implement interactive header navigation Implement header navigation component Mar 17, 2021
@ghost
Copy link
Author

ghost commented Mar 17, 2021

  • Implement non-interactive menu for mobile and large screens

This was done in #86

@ghost
Copy link
Author

ghost commented Mar 18, 2021

Implement a simple, interactive menu for mobile screens

This step will implement the logic to show and hide the mobile hamburger menu. #172 will implement the content that resides inside of the dropdown menu, once this logic is in place.

@ghost
Copy link
Author

ghost commented Mar 23, 2021

Tailwind didn't indicate this clearly, but I needed to make use of both z-order and hidden classes to cause proper showing and hiding.

@ghost ghost mentioned this issue Mar 23, 2021
@ghost
Copy link
Author

ghost commented Mar 23, 2021

  • Implement a minimal, interactive menu for mobile screens

Completed in #209

@ghost
Copy link
Author

ghost commented Mar 26, 2021

  • Change large screen menu to use hidden instead of opacity to fix subtle issue with hidden menu capturing mouse events

Done within this PR

@ghost
Copy link
Author

ghost commented May 27, 2021

Attempt refactoring implementation to make use of headless-ui library (timebox this, create a follow-up issue if it turns out to be complex)

This appears to be partially or fully done in PR #385.

@ghost ghost assigned tmattio Jul 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant