Skip to content

fix: #16618#18315

Open
NanHH wants to merge 1 commit into
quasarframework:devfrom
NanHH:patch-1
Open

fix: #16618#18315
NanHH wants to merge 1 commit into
quasarframework:devfrom
NanHH:patch-1

Conversation

@NanHH

@NanHH NanHH commented Jun 11, 2026

Copy link
Copy Markdown

#16618 (comment)

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Documentation
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the dev branch (or v[X] branch)
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
  • It's been tested on a Cordova (iOS, Android) app
  • It's been tested on an Electron app
  • Any necessary documentation has been added or updated in the docs or explained in the PR's description.

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to start a new feature discussion first and wait for approval before working on it)

Other information:

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts Quasar’s IME/composition handling for masked inputs so that Chromium-based browsers correctly treat numeric keypad input under Chinese IME as an active composition session, preventing premature mask updates that can duplicate the last digit (issue #16618).

Changes:

  • Update useKeyComposition to mark qComposing for non-Firefox browsers when compositionupdate carries plain-text (e.data) content (digits/latin), not only CJK characters.
  • This causes QInput / QSelect to defer masked input processing until compositionend/change, avoiding mid-composition mask side effects.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants