[PR #39] [MERGED] fix: include padding in auto mode image export dimensions #58

Closed
opened 2026-03-03 15:57:34 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mrgoonie/winshot/pull/39
Author: @mrgoonie
Created: 12/13/2025
Status: Merged
Merged: 12/13/2025
Merged by: @mrgoonie

Base: devHead: goon


📝 Commits (1)

  • 8f07694 fix(output): include padding in auto mode dimensions calculation

📊 Changes

2 files changed (+8 additions, -2 deletions)

View changed files

📝 frontend/dist/index.html (+1 -1)
📝 frontend/src/App.tsx (+7 -1)

📄 Description

Fixed image cropping issue when using Copy, Quick Save, or Save As operations. The calculateOutputDimensions function in App.tsx was not including padding in auto mode, causing exported images to be smaller than displayed on screen.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mrgoonie/winshot/pull/39 **Author:** [@mrgoonie](https://github.com/mrgoonie) **Created:** 12/13/2025 **Status:** ✅ Merged **Merged:** 12/13/2025 **Merged by:** [@mrgoonie](https://github.com/mrgoonie) **Base:** `dev` ← **Head:** `goon` --- ### 📝 Commits (1) - [`8f07694`](https://github.com/mrgoonie/winshot/commit/8f07694f9e74f6d6877781b60220c47b4bbdd5ac) fix(output): include padding in auto mode dimensions calculation ### 📊 Changes **2 files changed** (+8 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `frontend/dist/index.html` (+1 -1) 📝 `frontend/src/App.tsx` (+7 -1) </details> ### 📄 Description Fixed image cropping issue when using Copy, Quick Save, or Save As operations. The calculateOutputDimensions function in App.tsx was not including padding in auto mode, causing exported images to be smaller than displayed on screen. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 15:57:34 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/winshot#58
No description provided.