Fixes - #9
Merged
Merged
Conversation
Send complete search result sets through one selection operation while preserving support for single-element callers. Read transformed quad and fixed-position geometry before mutating the document, reuse that snapshot for handles, labels, and rotation controls, and append all generated overlays through one document fragment. This removes interleaved layout reads and per-overlay body insertions. Add a Puppeteer performance harness covering 20/40/80/160-node search selections and CDP layout/style metrics. In the controlled benchmark, 80-node selection improved from 1552 ms to 325 ms and 160-node selection from 4839 ms to 598 ms, with layout passes reduced from three per node to one total. Cover history behavior explicitly: bulk selection itself remains outside undo history, while a multi-element margin edit is recorded, undone, and redone as one batch.
Schedule a short submenu close when the pointer leaves either the parent action or its child popover, and cancel it when the pointer crosses between them. Preserve keyboard navigation by keeping the submenu open while its parent or contents retain focus, and clear pending timers during menu teardown. Update the browser regression coverage to verify that child menus collapse after moving away, the root actions menu remains anchored, and crossing the parent/submenu gap does not close the child.
Defer Enter submissions until the browser has committed the highlighted datalist value, allowing keyboard-selected commands to execute correctly. Read the current input value on every submission so refocusing the command box and pressing Enter repeats the command. Cancel pending idle queries to avoid duplicate execution, clean up timers and listeners on deactivation, and cover both flows with a browser regression test.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.