Skip to content

chore(viewport): cover unsupported zoom range keys - #416

Merged
coryrylan merged 1 commit into
mainfrom
topic/upkeep/coverage/zoom-range-keyboard
Oct 14, 2026
Merged

coryrylan merged 1 commit into
mainfrom
topic/upkeep/coverage/zoom-range-keyboard

Conversation

@cursor

@cursor cursor Bot commented Oct 14, 2026

Copy link
Copy Markdown

Selected task

Coverage task selected because: “1 uncovered lines, 98.21% lines and 87.03% branches, the smallest gap below the 90% threshold.”

Measurement

  • Lines: 55/56 (98.21%) → 56/56 (100%)
  • Branches: 47/54 (87.03%) → 50/54 (92.59%)
  • Statements: 70/73 (95.89%) → 72/73 (98.63%)
  • No other file’s coverage decreased.

No public API changed. This only verifies that an unrelated key leaves an explicitly stepped zoom range unchanged. Remaining defensive branches and production code were deliberately left alone.

Validation: viewport lint, unit, types, axe, SSR, Lighthouse, visual, and final repository-wide CI all pass.

Open in Web View Automation 

Verifies unrelated keys do not change an explicitly stepped zoom range, raising line coverage to 100% and branch coverage to 92.59%. No behavior change.

Signed-off-by: Cursor Agent <cursoragent@cursor.com>

Co-authored-by: Cory Rylan  <coryrylan@users.noreply.github.com>
@coryrylan coryrylan added the upkeep label Oct 14, 2026 — with Cursor
@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/elements/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: dfb5f0a1-4fd7-4a20-b9b3-c36a9b39c967

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@github-code-quality

Copy link
Copy Markdown

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/vitest

The overall line coverage in commit 79802da in the topic/upkeep/coverag... branch remains at 99%, unchanged from commit 7ab75d4 in the main branch.

@coryrylan
coryrylan marked this pull request as ready for review October 14, 2026 12:08
@coryrylan
coryrylan merged commit 0ddc7b0 into main Oct 14, 2026
18 checks passed
@coryrylan
coryrylan deleted the topic/upkeep/coverage/zoom-range-keyboard branch October 14, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants