Skip to content

Context menu visual action hover and "go back to previous submenu" button - #2272

Open
ada-tv wants to merge 5 commits into
overte-org:masterfrom
ada-tv:contextMenuHover
Open

Context menu visual action hover and "go back to previous submenu" button#2272
ada-tv wants to merge 5 commits into
overte-org:masterfrom
ada-tv:contextMenuHover

Conversation

@ada-tv

@ada-tv ada-tv commented May 16, 2026

Copy link
Copy Markdown
Collaborator
image image

@ada-tv ada-tv added needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested user experience Making the game easier and more fun to use, or changing unexpected and confusing behavior labels May 16, 2026

@zedwick zedwick left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just like a couple of comments added, but otherwise the code looks good to me!

Comment thread scripts/communityScripts/contextMenu.js
Comment thread scripts/communityScripts/contextMenu.js
@ada-tv
ada-tv force-pushed the contextMenuHover branch from aeb3aaf to ec807f3 Compare June 19, 2026 11:19
@ada-tv

ada-tv commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator Author

i've documented the globals and i've made the hover highlight less eye-searing

Comment thread scripts/communityScripts/contextMenu.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested user experience Making the game easier and more fun to use, or changing unexpected and confusing behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants