-
Notifications
You must be signed in to change notification settings - Fork 1
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
107 Broken Members Dropdown #115
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concise bugfix, detailed description, great naming and commit conventions, AND implementing additional error checking on your own initiative? Dang this is a really great PR. See one place where I think you can use tailwind rather than inline CSS (references provided!) - fix that and I'm happy to merge!
(P.s. I altered some confusing code from 2023 that was tangentially-related to your PR so do a git pull
on this branch before changing anything 😄)
next/components/header/header.tsx
Outdated
flexDirection: "column", // Stack the years vertically | ||
alignItems: "center", // Center the text horizontally | ||
padding: "0.25rem 0.5rem", // Adjust padding to fit the text better | ||
}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Convert these three to tailwind classes ^
https://tailwindcss.com/docs/flex-direction
https://tailwindcss.com/docs/padding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm thanks for those changes :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@kk7-py yo! Two approvals means you've got the go-ahead to merge this into main, GJ :D |
Context
Closes #107
What Changed?
How To Review
Hover over the members heading on the website to see the hidden dropdown menu, and check if the years are aligned vertically and in descending order.
Checkout
next/components/header/header.tsx
to view the changes.
Review implemented sorting logic and styles