Skip to content

feat: vi-mode indicator - #30

Merged
youssefadly237 merged 1 commit into
lryffel:mainfrom
youssefadly237:toolbar_indicator
Mar 2, 2026
Merged

youssefadly237 merged 1 commit into
lryffel:mainfrom
youssefadly237:toolbar_indicator

Conversation

@youssefadly237

@youssefadly237 youssefadly237 commented Aug 19, 2025 •

Copy link
Copy Markdown
Collaborator

this PR is implements registerPlaceholder, setPlaceholderValue from Xournal's Lua API to establish mode indicator for vi-xournalapp, without breaking older versions of Xournal++

The tool gets added to the toolbar like any other tool

Screenshot_20250819_233854

changes live by changing the mode

Screenshot_20250819_233932

closes #2

@youssefadly237
youssefadly237 marked this pull request as draft August 19, 2025 20:50
@youssefadly237
youssefadly237 marked this pull request as ready for review September 2, 2025 19:30
@youssefadly237

Copy link
Copy Markdown
Collaborator Author

xournalpp/xournalpp#6601 is merged

@lryffel

lryffel commented Sep 5, 2025

Copy link
Copy Markdown
Owner

Looks great! Do you know when the commit will make it into a (non-nightly) release?

@youssefadly237

Copy link
Copy Markdown
Collaborator Author

not sure tbh, but it should not affect the non nightly users

@youssefadly237

Copy link
Copy Markdown
Collaborator Author

this needs to be refactored will be done by tomorrow

@youssefadly237
youssefadly237 merged commit 1e5a3cb into lryffel:main Mar 2, 2026
1 check passed
@youssefadly237
youssefadly237 deleted the toolbar_indicator branch March 2, 2026 23:38
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.

Mode indicator

2 participants