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

Kaisa/refactor/377 refactor development page to use new sidebar #386

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

kaisahakola
Copy link
Contributor

📄 Pull Request Overview

closes #377

🔧 Changes Made

  1. Updated the hero development page to use the new DropdownWrapperV2 sidebar.

  2. Set the sidebars min-width a bit wider so longer words fit better when screen size reduces.


Checklist Before Submission

  • Functionality: I have tested my code, and it works as expected.
  • Debugging: No console.log() or other debugging statements are left.
  • Clean Code: Removed commented-out or unnecessary code.

Prevents the arrow icons to not hide under longer words
Update it to render the correct NavMenuWithDropdowns based on screen size. Also set min-width to 250px so dropdown is wide enough in smaller screens in collapsed state.
Copy link
Collaborator

@Jonroi Jonroi left a comment

Choose a reason for hiding this comment

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

Just a reminder for me... Re-request once the discussed changes are complete.

Primary color when closed and secondary color when opened. Also add some styling: cursor pointer for the titles and a hover color change.
Remove underline from non active subdropdowns and remove arrow icons from subdropdowns
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