Skip to content

Conversation

@omarArm
Copy link
Collaborator

@omarArm omarArm commented Oct 28, 2025

Fixes

Changes

  • Added a context menu to the Live Watch window
  • Added a copy expression command
  • Added a "Add to Live Watch" command that adds to Live Watch window from source files, text editors, variables window, and watch window
  • Showing data type of expression when hovering over it in Live Watch window
  • Added a Show in Memory Inspector command in the context menu of Live Watch window

P.S. the user would have to right click on a window item in order to view the context menu. If he/she/they right click on the empty window in the background, it will not open up the context menu. However, that is consistent with other windows in VSCode (watch window, variables window, etc.)

P.S. I updated the command title in the different context menus to Add to Live Watch instead of Add To Live Watch without updating the screenshots

Screenshots

Here is a screenshot of the commands available when clicking on the ... at the top right of the window
image

Here is a screenshot of the commands available if you right click in the window itself
image

Here is a screenshot of the command "Add to Live Watch" if you right click on any expression in VSCode text editor
image

The data type appears when hovering over an expression in the Live Watch window
image

Add to Live Watch appear in variables window
image

Add to Live Watch appear in watch window
image

Show a Live Watch expression in Memory Inspector
image

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

Adding copy expression command to context menu

Adding tests for copy expression

Adding a new mock for clipboard
@omarArm omarArm force-pushed the addContextMenuToLiveWatch branch from 072442f to 72352b6 Compare October 28, 2025 11:09
@omarArm omarArm requested a review from jreineckearm October 28, 2025 11:11
@qltysh
Copy link

qltysh bot commented Oct 28, 2025

Diff Coverage: The code coverage on the diff in this pull request is 90.2%.

Total Coverage: This PR will decrease coverage by 0.19%.

File Coverage Changes
Path File Coverage Δ Indirect
src/views/live-watch/live-watch.ts -1.1
🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@omarArm omarArm changed the title Add context menu to live watch Add context menu to live watch and contribute to text editor context menu for adding expressions to Live Watch window Oct 28, 2025
Copy link
Collaborator

@jreineckearm jreineckearm left a comment

Choose a reason for hiding this comment

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

LGTM and works nicely!

@jreineckearm jreineckearm merged commit 0176ba9 into main Oct 30, 2025
15 checks passed
@jreineckearm jreineckearm deleted the addContextMenuToLiveWatch branch October 30, 2025 16:12
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.

3 participants