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

Gauges: Fixing broken auto sizing #79926

Merged
merged 1 commit into from
Jan 2, 2024
Merged

Gauges: Fixing broken auto sizing #79926

merged 1 commit into from
Jan 2, 2024

Conversation

torkelo
Copy link
Member

@torkelo torkelo commented Jan 2, 2024

#76042 broke the auto sizing option (which is the default and used by almost all panels as manual sizing is very new).

#76216 also caused same issue with Gauge Panel.

  • Restore the old min dimensions for BarGaugePanel
  • Restore the old min dimensions for GaugePanel

I think there are more unfixed bugs here. The BarGaugePanel is showing the "sizing mode" option even when orientation is auto (Gauge panels do not).

It's a bit sad, if we had only tested the gdev panel test dashboard we would have seen a huge change, all panels getting scrollbars. We really need automated visual regression testing :)

Current main:
Screenshot 2024-01-02 at 10 37 07

How it looked before (should look):
Screenshot 2024-01-02 at 10 33 10

Fixes #79797

@torkelo torkelo requested review from a team and Eve832 as code owners January 2, 2024 09:44
@torkelo torkelo requested review from Develer, adela-almasan, Clarity-89, L-M-K-B and sunker and removed request for a team January 2, 2024 09:44
@torkelo torkelo changed the title Gauges: Fixing auto broken auto sizing Gauges: Fixing broken auto sizing Jan 2, 2024
@torkelo torkelo requested review from nmarrs and removed request for Eve832, adela-almasan, Clarity-89, L-M-K-B and sunker January 2, 2024 09:44
@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.3.x milestone Jan 2, 2024
@grafana-pr-automation grafana-pr-automation bot added area/panel/gauge area/frontend type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project labels Jan 2, 2024
@torkelo torkelo modified the milestones: 10.3.x, 10.2.x Jan 2, 2024
@torkelo torkelo removed type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project area/frontend area/panel/gauge labels Jan 2, 2024
@grafana-pr-automation grafana-pr-automation bot added type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project area/panel/gauge area/frontend labels Jan 2, 2024
@torkelo torkelo added backport v10.2.x and removed backport A backport PR labels Jan 2, 2024
@torkelo torkelo merged commit 9126dbc into main Jan 2, 2024
47 checks passed
@torkelo torkelo deleted the fixing-gauge-sizing branch January 2, 2024 15:55
@grafana-delivery-bot grafana-delivery-bot bot modified the milestones: 10.2.x, 10.3.x Jan 2, 2024
grafana-delivery-bot bot pushed a commit that referenced this pull request Jan 2, 2024
@nmarrs nmarrs mentioned this pull request Jan 2, 2024
3 tasks
nmarrs pushed a commit that referenced this pull request Jan 2, 2024
Gauges: Fixing broken auto sizing (#79926)

(cherry picked from commit 9126dbc)

Co-authored-by: Torkel Ödegaard <torkel@grafana.com>
@leeoniya leeoniya mentioned this pull request Jan 18, 2024
@summerwollin summerwollin modified the milestones: 10.3.x, 10.3.0 Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/frontend area/panel/gauge backport v10.2.x type/bug type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Grafana: Bar Gauge "Bar Size = Auto" broken in 10.2.3
4 participants