Skip to content

feat: add back link when jumping from key popover to a macro - #3045

Merged
mondalaci merged 1 commit into
masterfrom
feature/macro-back-from-key-action-popover
Aug 8, 2026
Merged

feat: add back link when jumping from key popover to a macro#3045
mondalaci merged 1 commit into
masterfrom
feature/macro-back-from-key-action-popover

Conversation

@mondalaci

Copy link
Copy Markdown
Member

Summary

  • When jumping to a macro from the key action popover (Jump to macro / Assign new macro), show a Back to link above Used on: using the same keymap ⭢ layer ⭢ key breadcrumb style.
  • Keeps jump-back navigation separate from the assignment list, as suggested in feat: show macro key assignments on the macro page #3002 (comment).
  • Reuses the shared <back-to/> query-param pattern from module settings.

Test plan

  • From a keymap key popover Macro tab, use Jump to macro and confirm a back link like Back to QWERTY for PC ⭢ Fn ⭢ G key appears above Used on:
  • Click the back link and confirm it returns to the same keymap/layer/key with the popover open
  • Use Assign new macro from the popover and confirm the same back link appears on the new macro page
  • Open a macro from the side menu (no jump) and confirm no back link is shown; Used on: still works when assignments exist
  • On the macro page with a back link, select/clear a macro action and confirm the back link remains
  • Navigate from keymap to module settings and confirm Back to [name] keymap still works

Made with Cursor

Show a keymap ⭢ layer ⭢ key breadcrumb above Used on so reverse navigation is separate from the assignment list.

Co-authored-by: Cursor <cursoragent@cursor.com>
@pcooke9

pcooke9 commented Aug 8, 2026

Copy link
Copy Markdown

Seems to work as expected in Win 11, thanks!

@mondalaci
mondalaci merged commit e76b4c0 into master Aug 8, 2026
6 checks passed
@mondalaci
mondalaci deleted the feature/macro-back-from-key-action-popover branch August 8, 2026 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants