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

Scrollbar displayed even when element is not overflowing #5282

Closed
obulat opened this issue Dec 17, 2024 · 5 comments · Fixed by #5285
Closed

Scrollbar displayed even when element is not overflowing #5282

obulat opened this issue Dec 17, 2024 · 5 comments · Fixed by #5285
Assignees
Labels
🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix good first issue New-contributor friendly help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend

Comments

@obulat
Copy link
Contributor

obulat commented Dec 17, 2024

Description

Setting overflow to scroll instead of auto makes Windows browsers display the scrollbar even if the content is not scrollable (browsers on MacOS hide it)

Reproduction

  1. Open the website on a Windows computer
  2. See scrollbars on the homepage and on external sources modal.

Screenshots

Image

Environment

  • OS: Windows

Additional context

To fix the issue, the following lines need to change:

<div class="main-content flex flex-grow flex-col overflow-y-scroll">

Replace overflow-y-scroll with overflow-y-auto.

max-height: calc(100vh - var(--modal-header-height, 0) - 6rem);
overflow-y: scroll;

Replace scroll with auto

@obulat obulat added good first issue New-contributor friendly help wanted Open to participation from the community 🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend labels Dec 17, 2024
@openverse-bot openverse-bot moved this to 📋 Backlog in Openverse Backlog Dec 17, 2024
@Anuj012004
Copy link
Contributor

Hi, I’d like to work on this issue. Is it okay if I take this up?
Please let me know if there are any guidelines or additional details.

@dhruvkb
Copy link
Member

dhruvkb commented Dec 17, 2024

Yes @Anuj012004! Please go ahead, I'll assign this to you.

@openverse-bot openverse-bot moved this from 📋 Backlog to 📅 To Do in Openverse Backlog Dec 17, 2024
@CoDinAIy
Copy link

CoDinAIy commented Dec 17, 2024

Hello, has this issue been resolved yet? I am able to make a PR immediately if you approve. @WordPress/openverse-maintainers

Thanks

@dhruvkb
Copy link
Member

dhruvkb commented Dec 17, 2024

@CoDinAIy thanks for your interest in this issue, and I appreciate you stepping back to give @Anuj012004 the chance to complete the task they've requested. You can look at the list of good first issues to see if there are any other issues that you can help with.

@openverse-bot openverse-bot moved this from 📋 Backlog to 🏗 In Progress in Openverse Backlog Dec 17, 2024
@openverse-bot openverse-bot moved this from 🏗 In Progress to ✅ Done in Openverse Backlog Dec 17, 2024
@CoDinAIy
Copy link

Hi @dhruvkb I shall take a look tomorrow, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix good first issue New-contributor friendly help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend
Projects
Status: ✅ Done
4 participants