fix: interactions in actions menu for mailboxes

[madbob]

Here I've fixed the interaction with the "Rename" option (mentioned in #9107) but also with "Add subfolder" (having the same problem). Both inputs are reverted to the static label when the menu is closed.

I've also added a "Saving" animation for "Add subfolder", equivalent to the one already existing for "Rename".

Fixes #9107

Summary by CodeRabbit

• Bug Fixes
  • Improved mailbox subfolder creation and rename interactions with clearer loading states.
  • Prevented duplicate actions while subfolders are being created or mailbox names are being updated.
  • Ensured input fields and menu state reset correctly after operations complete or are canceled.

[ChristophWurst]

Thanks!

[ChristophWurst]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

📝 Walkthrough

Walkthrough

NavigationMailbox replaces shared editing and saving flags with dedicated subfolder and rename state. Subfolder creation and rename flows now independently control loading, input availability, and reset behavior.

Changes

Mailbox actions

Layer / File(s)	Summary
Action state model src/components/NavigationMailbox.vue	Introduces dedicated subfolder and rename flags, updates action rendering, and resets these states when the menu collapses.
Subfolder creation flow src/components/NavigationMailbox.vue	Disables subfolder input during creation and restores the label state after completion.
Mailbox rename flow src/components/NavigationMailbox.vue	Tracks rename requests with renameSaving and clears the state when the request finishes or input opens.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: christophwurst, kesselb, gretad

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the mailbox actions menu interaction fix, which matches the main change.
Linked Issues check	✅ Passed	The changes address #9107 by letting the rename state close and revert when the menu collapses.
Out of Scope Changes check	✅ Passed	The Add subfolder saving-state updates are part of the stated mailbox actions menu fix and are not out of scope.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 2

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: ca30fafa-e9b1-4db3-893d-c29770d7eaa5

📥 Commits

Reviewing files that changed from the base of the PR and between e50eedf and 705da5e.

📒 Files selected for processing (1)

• src/components/NavigationMailbox.vue