Skip to content

fix(menu): make component actions work from the remaining actions menu - #9467

Merged
blizzz merged 2 commits into
mainfrom
fix/toolbar-remaining-actions
Oct 26, 2026
Merged

blizzz merged 2 commits into
mainfrom
fix/toolbar-remaining-actions

Conversation

@silverkszlo

@silverkszlo silverkszlo commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

📝 Summary

The bug

In the mobile app (or on mobile view in the browser) the menu/toolbar items Insert link, Insert attachment, Insert emoji and Nextcloud Assistant don't work.

To reproduce

  1. Open a text md document in mobile app or mobile view on desktop
  2. Click the 3-dot option menu and navigate to Insert emoji or Insert attachment or one of the other mentioned options
  3. Insert emoji gives "undefined" text on editor page, Insert attachment and the rest do nothing

Solution

Entries rendered by their own component (insert link, insert attachment, emoji picker, assistant) had no working action in the 3-dot menu. Clicking them now moves the entry into the menu bar and opens it there.

Also show the emoji picker on mobile devices and fix duplicate keys in the remaining actions menu.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI tools
  • The AI-generated content was reviewed, comprehended and tested by a human

@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Oct 26, 2026
@silverkszlo
silverkszlo force-pushed the fix/toolbar-remaining-actions branch from 6cc4a9a to fbdf12a Compare October 26, 2026 12:59
Comment thread src/components/Menu/ActionList.vue Resolved
@silverkszlo

silverkszlo commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator Author

failing playwright and cypress tests are unrelated

Signed-off-by: silver <s.szmajduch@posteo.de>
Assisted-by: ClaudeCode:claude-opus-5.5
Signed-off-by: silver <s.szmajduch@posteo.de>
Assisted-by: ClaudeCode:claude-opus-5.5
@blizzz
blizzz force-pushed the fix/toolbar-remaining-actions branch from fbdf12a to b8a4161 Compare October 26, 2026 19:49
@blizzz
blizzz merged commit 13bc015 into main Oct 26, 2026
65 checks passed
@blizzz
blizzz deleted the fix/toolbar-remaining-actions branch October 26, 2026 20:09
@silverkszlo

Copy link
Copy Markdown
Collaborator Author

/backport to stable35

@silverkszlo

Copy link
Copy Markdown
Collaborator Author

/backport to stable34

@silverkszlo

Copy link
Copy Markdown
Collaborator Author

/backport to stable33

@silverkszlo

Copy link
Copy Markdown
Collaborator Author

/backport to stable32

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants