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

[Bug]: Reports Month Dropdown Menus #1425

Closed
1 task done
carkom opened this issue Jul 30, 2023 · 5 comments
Closed
1 task done

[Bug]: Reports Month Dropdown Menus #1425

carkom opened this issue Jul 30, 2023 · 5 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@carkom
Copy link
Contributor

carkom commented Jul 30, 2023

Verified issue does not already exist?

  • I have searched and found no existing issue

What happened?

The box border has no radius on it.

Also the menus are showing horizontal scroll bars when they are no needed. I'm on a wide screen monitor with plenty of space for wider menus on this element. They should auto adjust the content so this doesn't happen.

image

What error did you receive?

No response

Where are you hosting Actual?

None

What browsers are you seeing the problem on?

Firefox, Chrome

Operating System

Windows 10

@carkom carkom added the bug Something isn't working label Jul 30, 2023
@MatissJanis MatissJanis added good first issue Good for newcomers help wanted Extra attention is needed labels Aug 26, 2023
@syukronrm
Copy link
Contributor

Hi @carkom, is this issue persist on your environment at the latest actual release?

@carkom
Copy link
Contributor Author

carkom commented Sep 22, 2023

Yes, still shows horizontal scroll bars. Checked other browsers. I've noticed the issue is not present in Chrome but only exists in firefox.

@miqh
Copy link
Contributor

miqh commented Oct 28, 2023

@carkom,

Applying scrollbar-gutter: stable to the ListboxList usages inside the common Select component should resolve the unwanted scroll bar overlay in Firefox and have it positioned like it would be in Chromium browsers.

<ListboxList style={{ maxHeight: 250, overflowY: 'auto' }}>

However, at this time of writing, a quick web search would suggest some ongoing implementation issues raised by users regarding that CSS property.

https://caniuse.com/mdn-css_properties_scrollbar-gutter

And if Safari is of concern, the property isn't supported at all. I don't have access to any devices to observe what currently happens there.

@lelemm
Copy link
Contributor

lelemm commented Oct 24, 2024

@carkom , is this persists?
tried on the latest firefox and chrome, no problems:

firefox:
image

chrome:
image

@carkom
Copy link
Contributor Author

carkom commented Oct 25, 2024

I just checked it on Firefox again and the issue has been resolved somewhere along the way. Thanks!

@carkom carkom closed this as completed Oct 25, 2024
@github-actions github-actions bot removed the help wanted Extra attention is needed label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

5 participants