Skip to content

fix(deskew): auto-oblique default and handle layout (#145) - #153

Merged
pablogventura merged 2 commits into
ScanTailor-Advanced:masterfrom
pablogventura:fix/deskew-oblique-regression-145
May 24, 2026
Merged

fix(deskew): auto-oblique default and handle layout (#145)#153
pablogventura merged 2 commits into
ScanTailor-Advanced:masterfrom
pablogventura:fix/deskew-oblique-regression-145

Conversation

@pablogventura

Copy link
Copy Markdown

Summary

Fixes deskew/oblique regression reported by @zvezdochiot on #145 after #152 merge:

  • Wire Automatic oblique correction checkbox to DefaultParams and Settings::pendingAutoOblique so auto deskew respects the global default (off by default).
  • Default per-page oblique mode to Manual in UI/UiData; stop pre-checking Oblique Auto in the panel.
  • Align deskew ImageView grid, crosshair, rotation/oblique arcs and handles to one content-area rect (scrollbar-aware) — fixes misaligned oblique levers in windowed mode.
  • DefaultParams XML: missing autoOblique attribute now defaults to off (matches constructor).
  • README: note that release AppImage targets Ubuntu 24.04 glibc (Provide valid Linux CI builds in Releases section as well #64).

Fixes #145 (near-term). Related #117.

Test plan

  • ctest — 5/5 passed locally (55 core tests)
  • Maintainer: new project → Deskew → with auto oblique off, no shear applied; handles aligned windowed vs maximized
  • @zvezdochiot: please re-test the Separate geometric distortions #145 scenario on this branch before we tag v1.2.1

Release note

Planned for v1.2.1 after re-test confirmation (not tagging in this PR).

…ailor-Advanced#145)

Connect Automatic oblique correction to DefaultParams and pending
auto-oblique on deskew reload. Default oblique mode to Manual in UI
and DefaultParams XML when the attribute is absent.

Align deskew ImageView grid, arcs, and handles to the same content
area rect (respecting visible scroll bars) so oblique levers stay
positioned in windowed mode.

Document AppImage glibc requirement (Ubuntu 24.04+) in README.
@pablogventura
pablogventura merged commit 91f5ba1 into ScanTailor-Advanced:master May 24, 2026
3 checks passed
pablogventura added a commit that referenced this pull request May 24, 2026
P0 bundle (#152) and deskew/oblique regression fix (#153).
@pablogventura
pablogventura deleted the fix/deskew-oblique-regression-145 branch May 24, 2026 20:50
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.

Separate geometric distortions

1 participant