Skip to content

feat: add configurable text_shadow option#94

Open
DennisGaida wants to merge 3 commits into
teuchezh:devfrom
DennisGaida:feature/configurable-text-shadow
Open

feat: add configurable text_shadow option#94
DennisGaida wants to merge 3 commits into
teuchezh:devfrom
DennisGaida:feature/configurable-text-shadow

Conversation

@DennisGaida
Copy link
Copy Markdown
Contributor

Description

For readability reasons, I sometimes would want more of a shadow behind the text and also behind the SVG icons. If for example a white cloud floats by sometimes the white text really is not very readable.

This adds a text_shadow config option (0-3, default 1) that scales the directional drop shadow applied to all text and SVG icons across the card. Uses CSS custom properties (--card-text-shadow, --card-icon-filter) set on the weather-card root so they cascade automatically into all sub-component shadow roots (weather-clock, weather-details, hourly-forecast, daily-forecast) without any JS manipulation.

Config: text_shadow: 1 # 0 = none, 1 = default, higher = stronger

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Translation

Related Issues

Fixes #

Checklist

  • Code follows project style
  • Self-reviewed code
  • Tested locally
  • bun run lint passed
  • bun run typecheck passed
  • bun run build successful
  • Commits follow Conventional Commits

Adds a text_shadow config option (0-3, default 1) that scales the
directional drop shadow applied to all text and SVG icons across the
card. Uses CSS custom properties (--card-text-shadow, --card-icon-filter)
set on the weather-card root so they cascade automatically into all
sub-component shadow roots (weather-clock, weather-details,
hourly-forecast, daily-forecast) without any JS manipulation.

Config: text_shadow: 1  # 0 = none, 1 = default, higher = stronger
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