Skip to content

fix(images): keep a turned picture's box to the size it is drawn at - #157

Merged
skjnldsv merged 1 commit into
mainfrom
fix/rotated-image-overflow
Oct 20, 2026
Merged

skjnldsv merged 1 commit into
mainfrom
fix/rotated-image-overflow

Conversation

@skjnldsv

Copy link
Copy Markdown
Contributor

Turning a tall picture once made the viewer scroll: the picture is drawn turned, but its box kept its old, tall size and overflowed. Margins now give the box the turned size, so it stays centred with no scrollbar.

Zooming in still overflows the same way, on main too. That's for another PR.

👾 This pull request was assisted by Claude Code, commits carry an Assisted-by trailer.

A quarter turn draws the picture across the box it is laid out in. A
tall picture turned kept its tall box, which overflowed the viewer:
a scrollbar showed and the picture sat off centre. Margins give the
box the turned size.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: John Molakvoæ <14975046+skjnldsv@users.noreply.github.com>
@skjnldsv skjnldsv added this to the 2.0.0 milestone Oct 20, 2026
@skjnldsv skjnldsv added type: bug 🐛 Something isn't working AI assisted 3. to review labels Oct 20, 2026
@skjnldsv skjnldsv self-assigned this Oct 20, 2026
@skjnldsv
skjnldsv merged commit 217e899 into main Oct 20, 2026
14 checks passed
@skjnldsv
skjnldsv deleted the fix/rotated-image-overflow branch October 20, 2026 10:54
@codecov

codecov Bot commented Oct 20, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.49%. Comparing base (d3af1c0) to head (d7ea95f).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
lib/components/Images.vue 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
- Coverage   92.54%   92.49%   -0.06%     
==========================================
  Files          51       51              
  Lines        4187     4185       -2     
  Branches      928      929       +1     
==========================================
- Hits         3875     3871       -4     
- Misses        289      291       +2     
  Partials       23       23              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

3. to review AI assisted type: bug 🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant