Skip to content

feat(card): artwork placement and size in the config.json card section (#94) - #400

Merged
rowkav09 merged 1 commit into
mainfrom
feat/card-artwork-config
Sep 24, 2026
Merged

rowkav09 merged 1 commit into
mainfrom
feat/card-artwork-config

Conversation

@rowkav09

Copy link
Copy Markdown
Member

Part of #94 (artwork placement and sizing).

The renderer already accepts artworkPosition, artworkWidth and artworkHeight, but config.json had no way to set them. This adds them to the card section:

  • artworkPosition: left or right
  • artworkWidth: 48 to 160
  • artworkHeight: 48 to 180

Values are checked the same way as the other card settings and passed through to the local card. Saving from the settings page keeps them (it used to rebuild the card section from only the fields it shows). The settings page controls come in a follow-up PR.

Tests: test/config-card.test.js covers save/parse, renderer mapping, bad values and settings-page saves.

#94)

card.artworkPosition (left/right), card.artworkWidth (48-160) and card.artworkHeight (48-180) are saved, validated and passed to the renderer. Saving from the settings page keeps them.
@github-actions

Copy link
Copy Markdown
Contributor

/mira pause

@github-actions github-actions Bot added the mira-paused Pause automatic Mira reviews on this pull request label Sep 24, 2026
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mira-paused Pause automatic Mira reviews on this pull request size:S

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant