fix(authoring): context menu flickers at previous position

[K1mmyn]

Issue

The main cause for the flicker is that, the current function which create the context menu check onmouseup if your cursor is on a piece of html which is apart of the context menu. This bug happens when you are holding right click and move your cursor off the context menu and release right click.

adjustedPosition is never cleared when the menu closes so the next open paints one frame at the previous menu's coordinates before the effect corrects it

Solution

Change from mouseup to mousedown because it is guaranteed that the mouse will start on top of the context menu due to negative offset

useLayoutEffect to adjust first before paint

Risk

NONE

Checklist

• Acceptance criteria met
• Continuous integration build passing

Summary by CodeRabbit

• Bug Fixes
  • Improved context menu click-away handling by responding when the mouse button is pressed, providing more consistent dismissal behavior.

[linear]

VPS-115

[coderabbitai]

📝 Walkthrough

Walkthrough

Changes

Context menu event handling

Layer / File(s)	Summary
Update click-away event binding frontend/src/components/ContextMenu/portal.jsx	clickAwayListener now removes and registers a document-level passive mousedown listener instead of a mouseup listener.

Estimated code review effort: 2 (Simple) | ~5 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly identifies the main change: fixing context-menu flickering at its previous position.
Description check	✅ Passed	The description includes the required Issue, Solution, Risk, and Checklist sections. It explains the stale adjustedPosition issue and the useLayoutEffect solution. Some checklist items remain unchecke…

Full details: Description check

Explanation

The description includes the required Issue, Solution, Risk, and Checklist sections. It explains the stale adjustedPosition issue and the useLayoutEffect solution. Some checklist items remain unchecked, but the description is mostly complete.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch vps-115-context-menus-noticeably-flicker-at-previous-position

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[GogaIV]

I mean i guesss

[harbassan]

Virtual.Patient.System.-.UoA.Mozilla.Firefox.2026-07-27.23-38-02.mp4

[GogaIV]

Screencast_20260727_235200.webm
it is working for me idk hartej, im on firefox browser
the other sort of flicker thats happening seems to be from the previous context menu location so could not be closing properly or smthn like that

[harbassan]

yeah i think that tiny flicker you have at the start of your video is the one im having all the time, where there is a render that happens at the previous location before a rerender at the new location.

[harbassan]

works