Skip to content

Fix ui/forms/input component for tag: :textarea#6174

Merged
tvdeyen merged 1 commit intosolidusio:mainfrom
chaimann:fix-textarea-component
Mar 3, 2025
Merged

Fix ui/forms/input component for tag: :textarea#6174
tvdeyen merged 1 commit intosolidusio:mainfrom
chaimann:fix-textarea-component

Conversation

@chaimann
Copy link
Copy Markdown
Contributor

@chaimann chaimann commented Mar 3, 2025

Fixes #6167

Summary

From MDN:

In HTML, the initial content of a <textarea> is specified between its opening and closing tags, not as a value attribute.

Therefore we must set content, rather than passing value for text areas.

Checklist

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/textarea
In HTML, the initial content of a <textarea> is specified
between its opening and closing tags, not as a `value`
attribute.

Therefore we must set `content`, rather than passing `value` for
 text areas.
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.81%. Comparing base (43b4607) to head (6b48128).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6174   +/-   ##
=======================================
  Coverage   88.81%   88.81%           
=======================================
  Files         835      835           
  Lines       18156    18158    +2     
=======================================
+ Hits        16125    16127    +2     
  Misses       2031     2031           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@chaimann chaimann marked this pull request as ready for review March 3, 2025 18:21
@chaimann chaimann requested a review from a team as a code owner March 3, 2025 18:21
Copy link
Copy Markdown
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏🏻

@tvdeyen tvdeyen added backport-v4.4 Backport this pull-request to v4.4 backport-v4.5 Backport this pull-request to v4.5 labels Mar 3, 2025
@tvdeyen tvdeyen merged commit 7d8a10d into solidusio:main Mar 3, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 3, 2025

💔 All backports failed

Status Branch Result
v4.4 An unhandled error occurred. Please see the logs for details
v4.5 An unhandled error occurred. Please see the logs for details

Manual backport

To create the backport manually run:

backport --pr 6174

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@fthobe
Copy link
Copy Markdown
Contributor

fthobe commented Mar 3, 2025

@chaimann thank you so much :)

@tvdeyen
Copy link
Copy Markdown
Member

tvdeyen commented Mar 6, 2025

💚 All backports created successfully

Status Branch Result
v4.5

Questions ?

Please refer to the Backport tool documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v4.4 Backport this pull-request to v4.4 backport-v4.5 Backport this pull-request to v4.5 changelog:solidus_admin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[New Admin] Text Area jscript is not visualizing data correctly

3 participants