-
Notifications
You must be signed in to change notification settings - Fork 486
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
fix(kit): improve caret management for InputNumber
on step action
#10405
Conversation
Pull request was closed ✔️All saved screenshots (for current PR) were deleted 🗑️ |
Visit the preview URL for this PR (updated for commit 14c5dfb): https://taiga-previews-demo--pr10405-input-number-caret-on-ste-91vzioyh.web.app (expires Wed, 19 Feb 2025 12:47:54 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 73dddc3c665194f3e11f18c16aeb71af4c289c37 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10405 +/- ##
==========================================
- Coverage 66.04% 66.00% -0.04%
==========================================
Files 1248 1248
Lines 16351 16360 +9
Branches 2336 2378 +42
==========================================
Hits 10799 10799
+ Misses 5393 5294 -99
- Partials 159 267 +108
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
BundleMonFiles updated (1)
Unchanged files (4)
Total files change +81B +0.01% Groups updated (1)
Final result: ✅ View report in BundleMon website ➡️ |
Playwright test results
Details
Failed testschromium › tests/kit/input-number/input-number.pw.spec.ts › InputNumber › API › [step] prop › caret position on step action › Keeps caret position on step › via button Flaky testschromium › tests/core/textarea/textarea.pw.spec.ts › Textarea › line break text Skipped testswebkit › tests/addon-commerce/input-card-group.pw.spec.ts › InputCardGroup › Examples › input card grouped with validation |
Previous behavior
Case 1. Unstable caret position
turbulent-caret.mov
Case 2. Caret is always placed to the end on step button / keyboard arrow
caret-shift.mov