Repository navigation
Presenter View: resizable section rail with slide previews - #1564
MisterSolon wants to merge 10 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Tip 🎯 Customize your risk assessments
✅ Automatic approvals, with your rules
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
See All Slides stopped at 48 slides and ignored sections. The grid now lists every slide, scrolls with fixed-size readable thumbnails, renders only the cells in view and keeps a bounded texture cache. Section tabs filter it (All slides / section / Other slides), with the current slide's section selected when it opens; the same grid works in Presenter View. A "Keep All slides open" checkbox leaves the presenter grid up after a pick, so the presenter can jump again without reopening it; turning it off restores the classic current/next/notes layout. Keys: Page Up/Down, Home/End and the arrow keys scroll, Left/Right switch tabs, Enter goes to the first slide of the selected tab. Co-Authored-By: DeepSeek V4.1 Flash <noreply@deepseek.com>
- Presenter View Enter uses pick_presenter_slide, so Keep All slides open also works from the keyboard. - grid_slide_hit ignores points outside the visible grid, so a click in the title or tab strip cannot pick a partly scrolled cell. - Restore the slide-show reliance weight (8) and the 110 total in the parity table; only the checklist row was added, the weight did not change. - Regression tests for the Enter path and the clipped hit test. Co-Authored-By: DeepSeek V4.1 Flash <noreply@deepseek.com>
abb2ea5 to
8e66ad5
Compare
8e66ad5 to
200a231
Compare
Dependency and scope
Draft, stacked on #1544. Keep this PR as a draft until A is merged, then rebase onto the resulting main so only the section rail and optional persistent grid remain in its diff.
Summary
Add a resizable, collapsible section rail beside the native presenter's current slide, next slide and notes. Opening or folding a section leaves the audience slide unchanged; picking a preview navigates to that slide. The rail supports one or two previews per row, highlights the current section, and truncates long names with a tooltip.
The divider resizes the rail up to a third of the window while preserving room for the slide and notes. Clicking the grip hides or restores it. Only visible previews remain cached, with bounded texture dimensions and per-frame rendering work; folding or hiding the rail frees its previews.
This PR introduces Keep All slides open, moved out of #1544. It defaults off: selecting a native thumbnail by click or Enter closes the grid. When enabled, selection preserves the grid. Disabling it closes the grid while preserving expanded rail sections. Escape, G and presenter-window close still dismiss the grid. The remote web presenter is unchanged.
The rail and persistence option are DeckCraft additions; they do not increase PowerPoint parity scores. No document-model or file-format change. A's startup GPU fix and upstream asynchronous next-slide previews are preserved.
Validation
Validated source: 200a231. Local validation is separate from GitHub CI and maintainer review; GitHub checks need to run on the updated branch. B remains a dependent draft.