Skip to content

Conversation

@ariane-emory
Copy link
Owner

@ariane-emory ariane-emory commented Dec 12, 2025

DUMMY PR, IGNORE

Fixes sst#5088 by adding a new optional \`placeholderText\` theme color that allows
customizing the color of placeholder text in input fields and textareas.

Changes:
- Add \`placeholderText\` to ThemeColors type with fallback to \`textMuted\`
- Apply \`placeholderColor\` prop to <input> components in dialog-select
- Apply styled placeholder text to <textarea> components in dialog-prompt and prompt
- Update JSON schemas to include \`placeholderText\` property
- Maintain backward compatibility - existing themes work unchanged

The new field follows ariane-emory's suggestion and provides a minimal,
focused solution for theming placeholder text.
- Update TextareaOptions interface to include optional placeholderColor field
- Add _placeholderColor private field and getter/setter methods to TextareaRenderable
- Modify constructor to initialize placeholderColor with fallback to #666666
- Update applyPlaceholder method to use placeholderColor instead of hardcoded fg(#666666)
- Update OpenCode components to use new placeholderColor prop instead of t/fg template functions
- Remove unused t, fg imports from dialog-prompt and prompt components

This enables consistent placeholder color theming across all input components using the same prop-based API pattern
- Remove t, fg imports from DialogPrompt and Prompt components
- Change textarea components to use placeholderColor prop instead of t/fg template functions
- This leverages the new placeholderColor prop added to opentui Textarea component
- Provides consistent API pattern matching Input component's placeholderColor usage
@ariane-emory ariane-emory force-pushed the fix/placeholder-color-5088 branch from 4739143 to d74137b Compare December 12, 2025 17:32
@ariane-emory ariane-emory force-pushed the fix/placeholder-color-5088 branch 2 times, most recently from a88b2f0 to d74137b Compare December 13, 2025 07:31
@ariane-emory ariane-emory force-pushed the dev branch 2 times, most recently from 37df00d to 56452d8 Compare December 16, 2025 02:13
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.

1 participant