Skip to content

馃崚 fix(web): prevent scrolling from changing server port (backport of #1052) - #1061

Open
yanziz-nvidia wants to merge 1 commit into
release/1.4.xfrom
yanziz/cherry-pick-1052-release-1.4.x
Open

馃崚 fix(web): prevent scrolling from changing server port (backport of #1052)#1061
yanziz-nvidia wants to merge 1 commit into
release/1.4.xfrom
yanziz/cherry-pick-1052-release-1.4.x

Conversation

@yanziz-nvidia

Copy link
Copy Markdown
Collaborator

Summary

Cherry-picks #1052 into release/1.4.x.

  • render the CloudXR server port as a text field so mouse-wheel scrolling cannot step and persist a different port
  • retain the numeric virtual keyboard hint with inputmode="numeric"
  • preserve source provenance with cherry-pick -x from 9fba23c4a3bd5b6de732cac77a47f471fca25276

Validation

  • SKIP=check-copyright-year pre-commit run --all-files
  • npm test -- --runInBand (102 tests)
  • npm run build
  • npx prettier --check src/index.html
  • parsed src/index.html with JSDOM and verified portInput has type=text and inputMode=numeric

Signed-off-by: Yanzi Zhu <yanziz@nvidia.com>
(cherry picked from commit 9fba23c)
@coderabbitai

coderabbitai Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

鈿欙笍 Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 152fa5b1-48c5-4d0e-aa1c-665a5d5e257b

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

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

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