Skip to content

Store initial "automatic read-only" decision, so that actively edited Note doesn't transition to read-only #1918

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

perfectra1n
Copy link
Member

@perfectra1n perfectra1n commented May 12, 2025

When a Note triggers "automatic read-only" threshold but is still being edited, trigger readOnlyTemporarilyDisabled event so that the user's edits still go through while they're still editing the Note.

Upon refreshing the Note frontend, it will appropriately go into read-only mode.

Closes #1877

@perfectra1n perfectra1n requested a review from eliandoran May 12, 2025 19:10
Copy link

github-actions bot commented May 12, 2025

Folder/File Previous size New size Difference
/upload/TriliumNextNotes-Server-1918-merge-linux-x64.tar.xz 69.45MB +69.45MB (+100.00%)
TOTAL +69.45MB

…only threshold is triggered

feat(client): stop the cursor from going to the top of the note when note transitions to read-only but we're still editing

feat(client): stop the cursor from going to the top of the note when note transitions to read-only but we're still editing, take 2
@perfectra1n perfectra1n force-pushed the feat/note-edit-readonly-fix2 branch from 68ae7c9 to caccbf4 Compare May 13, 2025 22:03
@perfectra1n perfectra1n changed the title Trigger readOnlyTemporarilyDisabled when "automatic read-only" threshold is crossed for text Notes Store initial "automatic read-only" decision, so that actively edited Note doesn't transition to read-only May 13, 2025
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.

Data loss when Note triggers "automatic read-only" threshold
1 participant