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

Fix <TextField> should call toString instead of JSON.stringify for non-string values #9888

Merged
merged 1 commit into from
May 30, 2024

Conversation

slax57
Copy link
Contributor

@slax57 slax57 commented May 30, 2024

Rationale: JSON.stringify displays a 'technical' representation of an object, while toString is more optimized to display a string representation of its content (e.g. when it is an Array or a Date).

This also has the benefit of keeping the same behavior as RA v4.

@slax57 slax57 added RFR Ready For Review v5 labels May 30, 2024
@djhi djhi added this to the 5.0.0 milestone May 30, 2024
@fzaninotto fzaninotto merged commit b55c276 into next May 30, 2024
14 checks passed
@fzaninotto fzaninotto deleted the TextField-toString branch May 30, 2024 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review v5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants