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.

Sign up for GitHub

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

Jump to bottom

IgcDateTimeInput: setRangeText method does not update value #1075

Closed
ddaribo opened this issue Feb 21, 2024 · 0 comments · Fixed by #1077
Closed

IgcDateTimeInput: setRangeText method does not update value #1075

ddaribo opened this issue Feb 21, 2024 · 0 comments · Fixed by #1077
Assignees
@rkaraivanov
Labels
bug Something isn't working ✅ status: resolved

Comments

@ddaribo
Copy link
Contributor

ddaribo commented Feb 21, 2024

Description

The setRangeText method should replace the selected text in the control and re-apply the mask. While it does update the underlying input's value, on subsequent clicks, it can be seen that the igc-date-time-input's value has not been updated, and reverts to the previous one.

Steps to reproduce

  1. Call the setRangeText method on an IgcDateTimeInputComponent component, for example in the docs demo.

Result

date-time-input-issue

Expected result

The updated input's value to be preserved, like for the igc-mask-input or the igc-input:

igc-mask-input-correct
@ddaribo ddaribo added bug Something isn't working 🆕 status: new The issue is new and will be reviewed when somebody picks it up. labels Feb 21, 2024
rkaraivanov added a commit that referenced this issue Feb 22, 2024
@rkaraivanov
fix(date-time-input): setRangeText updates underlying value
565ca97 
Closes #1075
@rkaraivanov rkaraivanov added ✅ status: resolved and removed 🆕 status: new The issue is new and will be reviewed when somebody picks it up. labels Feb 22, 2024
@rkaraivanov rkaraivanov mentioned this issue Feb 22, 2024
Merged
ChronosSF pushed a commit that referenced this issue Feb 29, 2024
@rkaraivanov
fix(date-time-input): setRangeText updates underlying value (#1077)
7f730cf 
Closes #1075
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rkaraivanov rkaraivanov

Labels
bug Something isn't working ✅ status: resolved
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(date-time-input): setRangeText updates underlying value IgniteUI/igniteui-webcomponents
3 participants
@onlyexeption @rkaraivanov @ddaribo