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

UI Fix: Tables become horizontally scrollable after clicking "Select All" #3537

Merged
merged 5 commits into from
Dec 19, 2024

Conversation

Shub3am
Copy link
Contributor

@Shub3am Shub3am commented Dec 17, 2024

UI Fix: Tables become horizontally scrollable after clicking "Select All"

Description

Changed The Width of the Selector Container to 95% instead of full-width which causes it to overflow unnecessarily on larger screens.

Fixes #3509

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

Screenshots & recording

Screen.Recording.2024-12-17.at.12.01.26.PM.mov

Manual review steps

Tested Visually on various screen sizes of different types: mobile, tablet and desktop screens ( upto 2k )

Copy link

codeclimate bot commented Dec 17, 2024

Code Climate has analyzed commit 9147e95 and detected 0 issues on this pull request.

View more on Code Climate.

@Paul-Bob
Copy link
Contributor

Thanks for looking into this @Shub3am!

There was an issue with your original approach:

image

I just pushed a commit that fixes this issue ☝️

@Paul-Bob Paul-Bob merged commit 5a5127a into avo-hq:main Dec 19, 2024
1 check passed
@Paul-Bob
Copy link
Contributor

@Shub3am I added another commit that computes the full width minus 2.5rem, precisely matching the rem value taken up by ml-10.

Thanks again for submitting this contribution! I took over to ensure it gets included in a patch release today.

@Paul-Bob Paul-Bob added the Fix label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tables become horizontally scrollable after clicking "Select All"
2 participants