-
Notifications
You must be signed in to change notification settings - Fork 467
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
feat(kit): allow dynamic changes to TUI_NUMBER_FORMAT #6856
Conversation
Pull request was closed ✔️All saved screenshots (for current PR) were deleted 🗑️ |
@@ -149,7 +149,7 @@ describe('InputRange', () => { | |||
}); | |||
|
|||
it('does not rounds to nearest multiple of [quantum] until text field losses focus', () => { | |||
inputPORight.sendText('8'); | |||
inputPORight.sendText('8 лет'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps this is due to the removal of the initial calibration plugin from input-number
, everything seems to be ok in the demo
Visit the preview URL for this PR (updated for commit ba421bf): https://taiga-ui--pr6856-format-dynamic-yhycjcta.web.app (expires Fri, 01 Mar 2024 10:12:09 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 4b5ece1e114386f6a105425ef799091475b249eb |
BundleMonFiles updated (2)
Unchanged files (3)
Total files change -5B 0% Groups updated (1)
Final result: ✅ View report in BundleMon website ➡️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll review InputNumber
changes once other comments are addressed as it seems like it will change quite a bit.
projects/core/directives/number-format/number-format.directive.ts
Outdated
Show resolved
Hide resolved
5bb8741
to
3d5dd3e
Compare
projects/demo/src/modules/directives/copy-processor/examples/1/index.ts
Outdated
Show resolved
Hide resolved
a2ba22b
to
1d97c3e
Compare
5858e07
to
ba421bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope it's all well without the initial calibration plugin? :)
Closes #2900, #2578
BREAKING CHANGES: remove
TUI_NUMBER_FORMAT_OBSERVABLE
in favour ofTUI_NUMBER_FORMAT
,tuiFormatNumber
pipe requires async pipe now