Skip to content

refactor: move divina cover reader to native containers#725

Merged
everpcpc merged 2 commits intomainfrom
codex/refactor-divina-cover-native
Apr 11, 2026
Merged

refactor: move divina cover reader to native containers#725
everpcpc merged 2 commits intomainfrom
codex/refactor-divina-cover-native

Conversation

@everpcpc
Copy link
Copy Markdown
Owner

Problem

The DIVINA cover reader was still driven by a SwiftUI cover path, which left it exposed to stale loading state, unnecessary slot refreshes, and first-transition animation glitches that did not affect the native scroll and page-curl readers.

Approach

Replace the cover-mode presentation with native UIKit and AppKit container implementations that own deck state, gestures, and animations directly. Reuse shared native paged page-content views so cover, scroll, and page-curl paths render through the same lower-level page presentation code, then tighten slot updates so stable-state refreshes do not reconfigure unchanged slots.

Scope

  • replace the old SwiftUI DIVINA cover implementation with native cover containers on iOS and macOS
  • share native paged page-content views between cover and paged readers
  • fix stale loading indicator behavior and first-transition animation issues on iOS and macOS
  • bump the build version to 396

Validation

  • make format
  • make build

Replace the SwiftUI-driven DIVINA cover path with native UIKit and AppKit cover containers, shared paged page content views, and native slot management.

Fix stale loading state, reduce redundant slot refreshes, and correct first-transition animation issues on iOS and macOS.
@everpcpc everpcpc merged commit 212621e into main Apr 11, 2026
3 checks passed
@everpcpc everpcpc deleted the codex/refactor-divina-cover-native branch April 11, 2026 13:29
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.

1 participant