Skip to content

Fix/zoom fit and artist ids - #333

Merged
feralbureau merged 2 commits into
mainfrom
fix/zoom-fit-and-artist-ids
Oct 30, 2026
Merged

feralbureau merged 2 commits into
mainfrom
fix/zoom-fit-and-artist-ids

Conversation

@feralbureau

Copy link
Copy Markdown
Member

What

Why

How

Checklist

  • pnpm check passes
  • Follows AGENTS.md: no comments in code, files under 150 lines, imports only point down the FSD layers, no any
  • Screenshot or recording attached for UI changes
  • Tested in the running app, not only in unit tests
  • docs/PRIVACY.md updated if this changes what the app sends off the device
  • New strings added to en.json (other languages are optional)

…tist lookup

- auto zoom now goes below 1 (down to MIN_ZOOM) when the work area is smaller
  than the 1100x705 layout, e.g. 1080p at 150% scaling, so the player bar no
  longer ends up under the taskbar. the user's own zoom is kept separately and
  comes back when the window grows again; ctrl+0 resets to 1, capped by the fit.
- getArtist no longer falls back to searching YouTube by name: it opened
  whichever artist ranked first, often someone else.
- artist names without a catalog id render as plain text instead of links that
  navigated by name.
@feralbureau
feralbureau merged commit b4a6c6e into main Oct 30, 2026
3 of 4 checks passed
@feralbureau
feralbureau deleted the fix/zoom-fit-and-artist-ids branch October 30, 2026 12:08
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

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.

2 participants