Skip to content

Refactoring 2#67

Merged
ttbombadil merged 11 commits intomainfrom
ref
Mar 23, 2026
Merged

Refactoring 2#67
ttbombadil merged 11 commits intomainfrom
ref

Conversation

@ttbombadil
Copy link
Collaborator

No description provided.

…S2004)

- Extracted async paste handler to separate function
- Reduces nesting from 5 to 3 levels
- Maintains same functionality with clipboard read/write
- Fixes SonarQube S2004 issue in code-editor.tsx
…l-monitor

- S3735: Removed void operator statements from processAnsiCodes
- S7758: Changed String.fromCharCode to String.fromCodePoint
- Kept ESC_RE/ESC_K_RAW variables for regression test compliance
… with button element (S6847/S6845)

- Converted non-interactive div overlay to semantic button element
- Removed explicit tabIndex and role attributes (button has these naturally)
- Fixes accessibility violations S6847 and S6845
- Updated ref types from HTMLDivElement to HTMLElement
- Updated usePinPollingEngine type signature for compatibility
@ttbombadil ttbombadil merged commit dc71b83 into main Mar 23, 2026
4 checks passed
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.

1 participant