search

macjabeth / showbunny

Watch and track your favourite movies and tv shows.
https://showbunny.xyz/
MIT License
6 stars 12 forks source link

fixed scroll on desktop #51

Closed ghost closed 2 years ago

ghost commented 3 years ago

Scrolling was always possible on desktop but the scroll bar was set to width: 0 rendering it imperceptible

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/macjabeth/showbunny/5qN2Fkk5VJu7VggcqsK325ApMWTt
✅ Preview: https://showbunny-git-fork-lillian-kodi-feature-issue-49-macjabeth.vercel.app

macjabeth commented 3 years ago

Hi @lillian-kodi - thanks for submitting the PR! The scrolling width of 0 was actually set intentionally.

My goal was to get rid of the horizontal scrollbar and instead have buttons which the user can click to scroll forward and back on desktop.

Screen Shot 2021-09-13 at 11 26 46 PM

Whether the buttons are above the media posters as depicted in the design above, or put as a hover element towards the beginning and end of the list similar to Disney+ or Hulu, either is fine.

Screen Shot 2021-09-13 at 11 45 59 PM Screen Shot 2021-09-13 at 11 46 45 PM

If you'd like to try your hand at implementing this feature, that would be most welcome! 🙂

ghost commented 3 years ago

Okay, I understand. Yes, I would love to take on this problem. Please may I have this task assigned to me?