Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes #15014: fixed issue with input number loosing focus #15015

Merged

Conversation

rosenthalj
Copy link
Contributor

Fixed #15014

I created a new unit test (should model value) in inputnumber.spec.ts.

The following screenshot is the unit test results for inputnumber.spec before the fix, the new unit test fails.

Karma

The following screenshot is the unit test results for inputnumber.spec after the fix, the new unit test passes.

Karma

The following video is running the inputNumber demo with the PR changes and shows that issue 1) has been fixed.

inputNumberFixNoNullPointerException.mov

The following video is running the reproducer with the PR changes and shows that issue 2) has been fixed.

inputNumberFixFocusChangesModelFromNumberToString.mov

Copy link

vercel bot commented Mar 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Mar 11, 2024 8:00pm

@cetincakiroglu cetincakiroglu merged commit 9b0518e into primefaces:master Mar 13, 2024
3 checks passed
@cetincakiroglu
Copy link
Contributor

Hi @rosenthalj,

Thanks a lot for the effort and support!

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.

p-inputnumber: multiple problems when input number looses focus
2 participants