Skip to content

[1607] Too much whitespace on gauge on mobile + desktop - #2195

Open
n-lark wants to merge 1 commit into
mainfrom
1607-gauge-whitespace
Open

[1607] Too much whitespace on gauge on mobile + desktop#2195
n-lark wants to merge 1 commit into
mainfrom
1607-gauge-whitespace

Conversation

@n-lark

@n-lark n-lark commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Description

On tall cells (common on mobile), dial gauges drifted down, leaving a gap between the title and the arc. The SVG filled the whole cell while the arc stayed capped by width.

The fix, in UIGaugeDial.vue's resize(): the gauge is a vertical stack (title on top, SVG below). When the cell is taller than the arc needs, we stop the SVG from stretching to fill and size it to just the arc plus room for the 0/100 labels. The stack then sits at the top so the arc sits right under the title, and any spare height falls below instead of opening a gap above. Short/wide/square cells skip this and keep the old behavior. Covers both half and three-quarter dials.

See before + after pics

Half Gauge

Before

Screenshot 2026-08-17 at 12 21 01 PM

After

Screenshot 2026-08-17 at 12 50 55 PM

3/4ths Gauge

Before

Screenshot 2026-08-17 at 12 20 46 PM

After

Screenshot 2026-08-17 at 12 21 41 PM

Related Issue(s)

Resolves #1607
Relates to https://github.com/FlowFuse/engineering/issues/242

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@n-lark n-lark self-assigned this Aug 17, 2026
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.

Too much whitespace on gauge on mobile

1 participant