Skip to content

Add reading mode (View > Enter Reading Mode / Mode lecture)#72

Merged
mcgivrer merged 2 commits into
mainfrom
feature/reading_mode
May 13, 2026
Merged

Add reading mode (View > Enter Reading Mode / Mode lecture)#72
mcgivrer merged 2 commits into
mainfrom
feature/reading_mode

Conversation

@jib63
Copy link
Copy Markdown
Collaborator

@jib63 jib63 commented May 11, 2026

Entering reading mode goes full-screen, hides the menu bar and the editor pane, floats the project explorer in a small utility window above the preview, and hides the LLM chat and all other side panels. Exiting (via the top-right button or Escape) restores every panel and divider position to its previous state.

  • MarkNote.java: promote root/topBar to fields; add enter/exit reading mode logic with floating Stage for the project explorer; add View menu item; suppress JavaFX full-screen exit hint
  • pom.xml: fix exec plugin config (split JVM args, use ${java.home}, commandlineArgs + workingDirectory); add git.commit.id.abbrev fallback; enable native git for git-commit-id plugin
  • i18n (6 files): add menu.view.readingMode and reading.mode.exit keys

Entering reading mode goes full-screen, hides the menu bar and the
editor pane, floats the project explorer in a small utility window
above the preview, and hides the LLM chat and all other side panels.
Exiting (via the top-right button or Escape) restores every panel and
divider position to its previous state.

- MarkNote.java: promote root/topBar to fields; add enter/exit reading
  mode logic with floating Stage for the project explorer; add View menu
  item; suppress JavaFX full-screen exit hint
- pom.xml: fix exec plugin config (split JVM args, use ${java.home},
  commandlineArgs + workingDirectory); add git.commit.id.abbrev fallback;
  enable native git for git-commit-id plugin
- i18n (6 files): add menu.view.readingMode and reading.mode.exit keys

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jib63 jib63 requested a review from mcgivrer May 11, 2026 21:20
Comment thread pom.xml
Comment thread pom.xml
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Attention, le retour du "Mode Lecture" provoque quelques soucis lors de la restauration de la position et de l'état des panels.
Un bug est apparu, si on ferme l'Explorateur de projet en mode lecture, le contenu du panel disparaît et il est impossible de le ramener dans l'affichage standard.

… doc update

- Fix project explorer showing blank after hide/detach/reading mode: restore
  panel center in DetachedPanelTab.hideWithoutDocking(), stop DockingManager
  from overwriting the close handler, add afterReattach() → refresh() in
  ProjectExplorerPanel
- Add minimize/maximize toggle button (▾/▴) to BasePanel, hidden by default
  and enabled only in reading mode; collapses the floating explorer to its
  title bar using the actual rendered header height + OS chrome height
- Save and restore all split-divider positions (editor/preview ratio, docking
  horizontal/vertical splits) when exiting reading mode; defer setDividerPositions
  to Platform.runLater so it applies after all rebuildLayout calls complete
- Add Ctrl+Shift+P accelerator for Enter Reading Mode menu item
- Update authors to Frédéric Delorme/JB Meyer in all 6 i18n locales
- Update README and user-guide-en.md with Reading Mode section, keyboard
  shortcut, and feature description

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mcgivrer
Copy link
Copy Markdown
Owner

I've checked your proposal and verified localy, I do not see any linked issue (an existing issue on the previous version must be tracked about Restoring Panels positions and states error). I validated it.

@mcgivrer mcgivrer merged commit d9a45f8 into main May 13, 2026
1 check passed
@mcgivrer
Copy link
Copy Markdown
Owner

I'll keep the branch open until further info confirming no bug.

@jib63 jib63 deleted the feature/reading_mode branch May 13, 2026 14:58
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.

2 participants