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 #26442: Edit Contentlet Add Text Area and enhance Text Field #26446

Conversation

zJaaal
Copy link
Contributor

@zJaaal zJaaal commented Oct 16, 2023

🤖 Generated by Copilot at 10ae5dd

Summary

🎨🚧🧪

Refactored the edit content field component to use custom components for text fields and textarea fields. Updated the unit tests and styles for the edit content field and form components. Added new files for the text field and textarea components and their models.

Edit content fields
<input> and <textarea>
Custom components

Walkthrough

  • Create two new custom components, DotEditContentTextFieldComponent and DotEditContentTextAreaComponent, to handle the logic and styles for different field types in the edit content form (F8-F15, F16).

Screenshots

Screen.Recording.2023-10-17.at.6.23.47.PM.mov

@zJaaal zJaaal linked an issue Oct 16, 2023 that may be closed by this pull request
@zJaaal zJaaal marked this pull request as draft October 16, 2023 21:40
@zJaaal zJaaal marked this pull request as ready for review October 17, 2023 21:25
@zJaaal zJaaal requested a review from fmontes October 18, 2023 19:05
@fmontes
Copy link
Member

fmontes commented Oct 18, 2023

image

Copy link
Member

@oidacra oidacra left a comment

Choose a reason for hiding this comment

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

I left few comments.

@zJaaal zJaaal requested a review from fmontes October 20, 2023 13:21
@dotcms-sonarqube
Copy link

@fmontes fmontes merged commit eabbb0a into master Oct 20, 2023
@fmontes fmontes deleted the 26442-edit-content-render-textfield-and-textarea-field-to-the-form branch October 20, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Edit Content: Render TextField and TextArea Field to the Form
5 participants