-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
SQL snippets show scrollbars when it isn't needed #21550
Labels
.CSS
Difficulty:Easy
.Frontend
Priority:P3
Cosmetic bugs, minor bugs with a clear workaround
Querying/Native
The SQL/native query editor
Querying/Parameters & Variables
Filter widgets, field filters, variables etc.
.Reproduced
Issues reproduced in test (usually Cypress)
Type:Bug
Product defects
Milestone
Comments
paoliniluis
added
Type:Bug
Product defects
Priority:P3
Cosmetic bugs, minor bugs with a clear workaround
Querying/Native
The SQL/native query editor
.Frontend
labels
Apr 8, 2022
flamber
added
.CSS
Difficulty:Easy
Querying/Parameters & Variables
Filter widgets, field filters, variables etc.
labels
Apr 9, 2022
ariya
added a commit
that referenced
this issue
Apr 18, 2022
ariya
added a commit
that referenced
this issue
Apr 18, 2022
This was referenced Jan 11, 2024
This was referenced Feb 5, 2024
Merged
Open
This was referenced Apr 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
.CSS
Difficulty:Easy
.Frontend
Priority:P3
Cosmetic bugs, minor bugs with a clear workaround
Querying/Native
The SQL/native query editor
Querying/Parameters & Variables
Filter widgets, field filters, variables etc.
.Reproduced
Issues reproduced in test (usually Cypress)
Type:Bug
Product defects
Describe the bug
We show scrollbars for the SQL snippets even when it's not needed
Logs
None
To Reproduce
In a Metabase instance go to:
Expected behavior
If we don't need scrollbars, either horizontal or vertical, we shouldn't show them
Screenshots
Information about your Metabase Installation:
Severity
Cosmetic
Additional context
NA
The text was updated successfully, but these errors were encountered: