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

fix(styling): Grid Menu & Column Picker overflow in Firefox #845

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

ghiscoding
Copy link
Owner

@ghiscoding ghiscoding commented Dec 12, 2022

  • remove overflow-x for all browsers, we don't need a scroll to show at any point in time for the x coordinate
  • Firefox has issues when overflow (auto) shows the scroll, it shows up over the text content while Chrome seems to add necessary padding to avoid that problem, so we can add a fixed right padding for Firefox browser only, we don't want to make it too wide either because that would look weird

1. (before) in Firefox (column name is cut off and extra scrollback shows up in x axis)

image

image

2. (after) with extra padding on the right in Firefox only (x axis scrollbar is gone)

image

image

- remove overflow-x for all browsers, we don't need a scroll to show at any point in time for the x coordinate
- Firefox has issues when overflow (auto) shows the scroll, it shows up over the text while Chrome seems to add necessary padding to avoid that problem, so we can add a fixed right padding for Firefox only, we don't want to make it too wide because that would look funny
@ghiscoding ghiscoding changed the title fix(styling): Grid Menu & Col Picker overflow in Firefox fix(styling): Grid Menu & Column Picker overflow in Firefox Dec 12, 2022
@ghiscoding ghiscoding merged commit 9b0aef7 into master Dec 12, 2022
@ghiscoding ghiscoding deleted the bugfix/styling-grid-menu-padding-firefox branch December 12, 2022 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant