Skip to content

fix(menu): right-click on a link shows the menu; every row has an icon (#95) - #96

Merged
Maxaubert merged 3 commits into
mainfrom
fix/95-right-click-link
Sep 29, 2026
Merged

Maxaubert merged 3 commits into
mainfrom
fix/95-right-click-link

Conversation

@Maxaubert

@Maxaubert Maxaubert commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

Closes #95.

A right-click on a link opened the link and showed the menu at the same time. xterm's web-links addon activates on a click of any button.

  • Only a left click opens a link now.
  • The link menu gains Open link beside Copy link.
  • Every row of both menus has an icon, the terminal's and the tab's, as in Prism's Explorer (owner: "the items should have icons like in prism explorer"). The glyphs are Prism's FileMenuIcon paths and style; find, help and close are drawn to match.

Core change, so Prism gets it too through the automatic bump. The menu rows are this app's own; Prism's own right-click menu is unchanged. Core 0.16.1, app 0.19.1. These numbers overlap with the other open PRs, and whichever merges later gets bumped again.

Verified:

  • Test first: the selectionEdit e2e now asserts three things. A right-click on a link opens nothing. The menu reads Copy link, Open link. Open link opens it, and a left click still does. This failed on main (the link opened on the right-click, and there was no Open link row).
  • Unit tests: 937 pass. Lint and typecheck pass.
  • The e2e asserts that every row of the terminal's menu has its icon. .e2e-shots/term-menu-link.png was checked by eye.
  • The full e2e suite passes, lastTab included this time.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t

Maxaubert and others added 2 commits September 28, 2026 21:03
xterm's web-links addon activates on a click of any button. Only a left
click opens now, and the menu offers Open link beside Copy link.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t
@Maxaubert Maxaubert changed the title fix(links): a right-click on a link opens the menu, not the link (#95) fix(menu): right-click on a link shows the menu; every row has an icon (#95) Sep 28, 2026
# Conflicts:
#	core/package.json
#	package-lock.json
#	package.json
@Maxaubert
Maxaubert merged commit 0beafb1 into main Sep 29, 2026
2 checks passed
@Maxaubert
Maxaubert deleted the fix/95-right-click-link branch September 29, 2026 22:04
github-actions Bot pushed a commit that referenced this pull request Sep 29, 2026
#95) (#96)

* fix(links): a right-click on a link opens the menu, not the link (#95)

xterm's web-links addon activates on a click of any button. Only a left
click opens now, and the menu offers Open link beside Copy link.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t

* feat(menu): a glyph on every menu row, as in Prism's Explorer (#95)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FHHaWKR4M5QtW7Wecyuk4t

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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.

Right-clicking a link opens it instead of showing the menu

1 participant