Skip to content

fix(viewer): say when changes to a mind map are not saved - #422

Merged
AndyScherzinger merged 1 commit into
mainfrom
fix/readonly-format-notice
Oct 22, 2026
Merged

AndyScherzinger merged 1 commit into
mainfrom
fix/readonly-format-notice

Conversation

@AndyScherzinger

Copy link
Copy Markdown
Member

Refs #166

FreeMind (.mm) and XMind files open in the editor and can be edited, but cannot be saved. For these formats the editor hides the save button and the autosave checkbox, and it closes without asking about unsaved changes, so users only find out their edits are gone when they open the file again (see the comments on #166).

When a file in such a format opens, FilesMindMap.load() now shows the toast "Changes to {extension} files are not saved." for 10 seconds. This PR does not add saving in .mm format, which is the actual feature request in #166, so it only refers to the issue instead of closing it.

Tested: added a check to the existing Vitest case for read-only formats that the toast is shown. Vitest and ESLint pass. I have not yet tested it in a running Nextcloud. The new string will be translated through Transifex.

A backport to stable-0.1 follows.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

FreeMind and XMind files can be opened and edited, but not saved. The
editor hid the save controls and closed without asking, so users lost
their edits without being told. Show a notice when such a file opens.

Refs #166

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger added bug Something isn't working AI assisted This PR contains AI-assisted commits labels Oct 21, 2026
@AndyScherzinger AndyScherzinger added this to the v0.2.0 milestone Oct 22, 2026
@AndyScherzinger
AndyScherzinger merged commit 3d1a77f into main Oct 22, 2026
8 checks passed
@AndyScherzinger
AndyScherzinger deleted the fix/readonly-format-notice branch October 22, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant