Skip to content

[JEWEL-640] Display Placeholders for Undecorated TextField/TextArea #3111

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

Closed

Conversation

DanielSouzaBertoldi
Copy link
Collaborator

@DanielSouzaBertoldi DanielSouzaBertoldi commented Jul 1, 2025

Context

Currently our TextField/TextArea composables don't show placeholder for undecorated versions. This PR fixes this.

Main Changes

  • Updated both TextField and TextArea samples to also have an undecorated example.
  • Created a common TextAreaContentWrapper to remove code duplication between undecorated and decorated versions of TextArea.
  • Updated NoTextAreaDecorator composable to expect optional placeholder data (just like TextAreaDecorator) and added a Box so that we can show the placeholder on top of the TextArea area.
  • Created a UndecoratedTextFieldDecorationBox that also has a Box just like the TextArea version, where we can place the placeholder on top.

Evidences

Standalone

TextField

Context Evidence
Before
Screen.Recording.2025-07-01.at.11.24.08.mov
After
After (Dark)
Typing Something
Screen.Recording.2025-07-01.at.11.09.11.mov

TextArea

Context Evidence
Before
Screen.Recording.2025-07-01.at.11.23.29.mov
After
After (Dark)
Typing Something
Screen.Recording.2025-07-01.at.11.07.16.mov

IDE

TextField

Context Evidence
Before
Screen.Recording.2025-07-01.at.11.26.32.mov
After
After (Light)
Typing Something
Screen.Recording.2025-07-01.at.11.13.59.mov

TextArea

Context Evidence
Before
Screen.Recording.2025-07-01.at.11.26.06.mov
After
After (Light
Typing Something
Screen.Recording.2025-07-01.at.11.13.29.mov

Release notes

Bug fixes

  • TextField and TextArea marked as undecorated now show placeholders (if they have any).

Copy link
Collaborator

@rock3r rock3r left a comment

Choose a reason for hiding this comment

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

Nice job — just a z-order issue then we can ship

@rock3r
Copy link
Collaborator

rock3r commented Jul 7, 2025

@DanielSouzaBertoldi please add the release notes :)

@rock3r
Copy link
Collaborator

rock3r commented Jul 7, 2025

Ready to merge

intellij-monorepo-bot pushed a commit that referenced this pull request Jul 8, 2025
closes #3111

GitOrigin-RevId: fcba8a712db460d321f2428f13eaac3ee2d4ebca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants