compass icon indicating copy to clipboard operation
compass copied to clipboard

Hide grid scrollbar if not actively scrolling

Open tyler-dane opened this issue 1 year ago • 2 comments

To help with the minimal aesthetic, hide the grid's scrollbar until actively scrolling

scrollbar

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

tyler-dane avatar Nov 01 '24 12:11 tyler-dane

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?

MrPand-21 avatar Jan 24 '25 19:01 MrPand-21

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

tyler-dane avatar Jan 25 '25 13:01 tyler-dane

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 avatar May 28 '25 15:05 Iwueseiter

@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)

tyler-dane avatar May 28 '25 18:05 tyler-dane

@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.

Iwueseiter avatar May 28 '25 20:05 Iwueseiter

Unassigning and moving back to Ready due to inactivity

tyler-dane avatar Jun 06 '25 12:06 tyler-dane

https://github.com/user-attachments/assets/73477f4a-56e6-4eaf-adb2-ebbc3745ceba

@tyler-dane @that-one-arab is like that?

Amumug avatar Aug 01 '25 12:08 Amumug

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 avatar Aug 06 '25 12:08 tyler-dane

@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 avatar Aug 06 '25 14:08 Amumug

@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 avatar Aug 06 '25 15:08 that-one-arab

@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!

Amumug avatar Aug 06 '25 19:08 Amumug

@that-one-arab is no problem if i work on this milestone?

Image

Amumug avatar Aug 07 '25 12:08 Amumug

@Amumug Not at all, this milestone has a number of issues, find an issue you want to work on and tag us in it.

that-one-arab avatar Aug 07 '25 14:08 that-one-arab