Skip to content

[WIP] Optimize synchronous DOM inserts on pad load#99

Merged
JohnMcLear merged 5 commits into
mainfrom
copilot/optimize-dom-inserts
May 14, 2026
Merged

[WIP] Optimize synchronous DOM inserts on pad load#99
JohnMcLear merged 5 commits into
mainfrom
copilot/optimize-dom-inserts

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026 •
edited
Loading

  • Inspect current implementation, tests, and recent CI/workflow failures to confirm scope
  • Implement lazy population of the special characters list on first modal open with a bounded curated range and a single DOM append
  • Fix character insertion click handler so it does not throw require is not defined in browser context
  • Add/adjust focused frontend tests to cover lazy population and successful character insertion
  • Run targeted lint/tests for changed files and capture a UI screenshot of the modal behavior
  • Run parallel validation (Code Review + CodeQL), address any findings, and report final status

Copilot AI linked an issue May 13, 2026 that may be closed by this pull request
Pathological page load: 5,001 synchronous DOM inserts on every pad #87
Closed
Copilot AI requested a review from JohnMcLear May 13, 2026 10:50
@JohnMcLear
fix: capture ace via postAceInit for special chars
ea50040 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@JohnMcLear JohnMcLear marked this pull request as ready for review May 14, 2026 10:14
@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented May 14, 2026

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@JohnMcLear JohnMcLear merged commit 655dfd9 into main May 14, 2026
3 checks passed
@JohnMcLear JohnMcLear deleted the copilot/optimize-dom-inserts branch May 14, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JohnMcLear JohnMcLear Awaiting requested review from JohnMcLear

Assignees

@JohnMcLear JohnMcLear

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Pathological page load: 5,001 synchronous DOM inserts on every pad

2 participants

@JohnMcLear