New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#8075 ensure we capture events in selection menu handlers #8096
#8075 ensure we capture events in selection menu handlers #8096
Conversation
@@ -48,12 +48,6 @@ export const selectionMenuActionRegistry = new ActionRegistry(); | |||
|
|||
let selectionMenu: Nullishable<HTMLElement>; | |||
|
|||
const onMousedownHide = (event: MouseEvent) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved this function closer to where it's used for better code organization
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8096 +/- ##
==========================================
- Coverage 73.20% 73.19% -0.01%
==========================================
Files 1310 1310
Lines 40730 40732 +2
Branches 7567 7569 +2
==========================================
- Hits 29816 29815 -1
- Misses 10914 10917 +3 ☔ View full report in Codecov by Sentry. |
When the PR is merged, the first loom link found on this PR will be posted to |
* get sidebar state in background * starter mods sidebar panels are closed on install * fix imports * add sidebarStorage to strictNullChecks * adds tests for sidebarStorage * adds more test coverage * add tests for getModComponentIdsForModComponentDefinitions * renaming * update factory to support overriding starterBrickType * revert factory changes * lint fixes * add troubleshooting section to e2e tests readme (#8095) * ensure we capture events in selection menu handlers (#8096) * pr-related fixes * Update src/background/starterMods.ts Co-authored-by: Ben Loe <below413@gmail.com> * fix types --------- Co-authored-by: Eduardo Fungairino <eduardo@pixiebrix.com> Co-authored-by: Ben Loe <below413@gmail.com>
What does this PR do?
Demo
Checklist