Skip to content

feat(page_layout): inline source DPI in Margins panel (#93) - #151

Merged
pablogventura merged 1 commit into
ScanTailor-Advanced:masterfrom
pablogventura:feature/issue-93-margins-fix-dpi
May 24, 2026
Merged

feat(page_layout): inline source DPI in Margins panel (#93)#151
pablogventura merged 1 commit into
ScanTailor-Advanced:masterfrom
pablogventura:feature/issue-93-margins-fix-dpi

Conversation

@pablogventura

Copy link
Copy Markdown

Summary

  • Adds Source DPI preset combo + X/Y fields to the Margins panel (issue [feature request] "Fix DPI" in "Margins" panel #93).
  • Commits valid DPI on editingFinished; keeps keyboard focus on DPI fields when switching pages.
  • Renames the dialog launcher to Fix all… (Tools menu and bulk Fix DPI dialog unchanged).

Test plan

  • Open project with mixed/invalid source DPI; edit inline in Margins → thumbnails and margin mm values update after reload.
  • Focus X DPI field → Page Down / next thumbnail → focus stays on X DPI, value updates per page.
  • Fix all… still opens the full Fix DPI dialog for bulk edits.
  • Save/reopen project → inline-edited DPI persisted in project XML.

Closes #93

…lor-Advanced#93)

Add Source DPI combo and X/Y fields to the Margins panel so users can fix
per-image DPI without opening the bulk Fix DPI dialog. Values commit on
editingFinished; keyboard focus is preserved when switching pages while a
DPI field is active. The existing button opens Fix all… for bulk edits.
@pablogventura
pablogventura force-pushed the feature/issue-93-margins-fix-dpi branch from 875f8fa to 0b9ccf9 Compare May 24, 2026 02:19
@pablogventura
pablogventura merged commit 7f210ae into ScanTailor-Advanced:master May 24, 2026
3 checks passed
@pablogventura
pablogventura deleted the feature/issue-93-margins-fix-dpi branch May 24, 2026 02:23
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.

[feature request] "Fix DPI" in "Margins" panel

1 participant