Hide grid scrollbar if not actively scrolling
To help with the minimal aesthetic, hide the grid's scrollbar until actively scrolling
Implementation Guidance
Reference the CSS from the sidebar (#164), which successfully hides the scrollbar if not actively scrolling
Acceptance Criteria
- [ ] Scrollbar is hidden when the mouse is not over the main grid
- [ ] Scrollbar appears when the mouse is over the main grid
I can work on that
By the way would you consider using NextJs for this project or is there a specific reason for sticking to purely React without any other meta-frameworks?
Thanks for your willingness to help out, @MrPand-21. Unfortunately this is a Q3 issue, so it won't be worked on until after June 2025 at the earliest. Please find an issue with the Q1 or Q2 milestone instead. I just created a new view to make that easier to do: https://github.com/orgs/SwitchbackTech/projects/4/views/8
Here is the updated workflow for picking up issues: https://docs.compasscalendar.com/docs/contribute/#workflows
I also added a section to our roadmap that answers your question about NextJs: https://docs.compasscalendar.com/docs/roadmap#anti-priorities
I'm a frontend developer who is proficient with react, nextjs, Tailwind, JS, typeScript, integration ETC. I write efficient code that not only makes the frontend function as expected but also makes it appealing and highly responsive. Let me handle this task, as you'd be sure to get the best.
ETA: 2days
@Iwueseiter , thanks for volunteering and providing an estimate. Just assigned you and updated the dates.
I see you have some Tailwind and Next experience, but please just stick with the current tools we have for this issue (React and styled-components)
@Iwueseiter , thanks for volunteering and providing an estimate. Just assigned you and updated the dates.
I see you have some Tailwind and Next experience, but please just stick with the current tools we have for this issue (React and styled-components)
Got it! I'd get to work.
Unassigning and moving back to Ready due to inactivity
https://github.com/user-attachments/assets/73477f4a-56e6-4eaf-adb2-ebbc3745ceba
@tyler-dane @that-one-arab is like that?
Hey @Amumug - that looks pretty good. In fact, it's probably better than what I was originally thinking, which was to hide it entirely until the user was actively scrolling. That might have some a11y issues, though.
I saw you closed your PR. Were you not happy with the code there?
@tyler-dane I closed the PR because I had been waiting for a while and no one seemed available to review it. I figured it might not be planned for this quarter, so I decided to close it for now.
@Amumug Hey! Thanks for the work you're putting in! We truly appreciate having you as an active contributor. Unfortunately we sometimes get busy and reviews may not come as early as they should, but we do eventually review every single PR so don't worry! I will make sure to review your open PRs today or tomorrow, so feel free to re-open it if you think its ready!
@that-one-arab Thank you so much! I really appreciate your kind words. This is my first time contributing to open source, so I’m still learning and might make a lot of mistakes along the way — I apologize if I did anything wrong. I’ll go ahead and re-open the PR. Thanks again for your support!
@that-one-arab is no problem if i work on this milestone?
@Amumug Not at all, this milestone has a number of issues, find an issue you want to work on and tag us in it.