Skip to content

Fix: weather card edges in daylight and uneven System Summary column gaps - #466

Merged
SandObserver merged 1 commit into
mainfrom
fix/weather-card-edge-and-summary-gaps
Oct 16, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/weather-card-edge-and-summary-gaps

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • setCardAppearance(false) sets data-appearance="light" on the card instead of clearing it.
  • Add dark-theme card tokens for data-appearance="light" (white background, light-theme border and edges), so a card wider than its contain-fitted widget shows white at the sides behind the daytime Weather face.
  • Replace the flex gap in columns() with transparent inline borders, background-clip: padding-box and a negative container margin, so gaps stay equal under fractional widget zoom.

Testing
All green. Added a test for the column border spacing and updated the card appearance test.

Changelog

Fixed

  • Weather no longer shows dark bands at the card's sides in daytime with the dark theme.
  • System Summary chart columns have equal gaps.

Checklist

  • Stays within the project constraints: one container, no runtime dependencies, no build step (see CONTRIBUTING.md).
  • Tests added or updated for new logic, and the full suite, lint, and typechecks pass.
  • Docs updated if behavior or configuration changed.

- Mark a widget card light when the widget paints a light background, and
  give it the light card tokens in the dark theme.
- Space chart columns with transparent inline borders instead of flex gap.
@SandObserver SandObserver added the ui/ux UI, UX, and accessibility only fix label Oct 16, 2026
@SandObserver
SandObserver merged commit 4e8d3b4 into main Oct 16, 2026
6 checks passed
@SandObserver
SandObserver deleted the fix/weather-card-edge-and-summary-gaps branch October 16, 2026 19:03
stackyard-release Bot pushed a commit that referenced this pull request Oct 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui/ux UI, UX, and accessibility only fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant