Skip to content

feat(placeholder): draw deterministic hand-drawn squiggle - #2214

Draft
soulcramer wants to merge 2 commits into
mainfrom
feat/hand-drawn-text-placeholder
Draft

feat(placeholder): draw deterministic hand-drawn squiggle#2214
soulcramer wants to merge 2 commits into
mainfrom
feat/hand-drawn-text-placeholder

Conversation

@soulcramer

@soulcramer soulcramer commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator

This is a design proposition for a varaint to the current text placeholder.

The current modifier drew a solid pill add this variant make it a more "organic" it by drawing it with squiggles.

@soulcramer
soulcramer requested review from a team and SimonMarquis as code owners August 22, 2026 11:45
@soulcramer soulcramer added enhancement New feature or request component Related to a component LLM Mainly built by a LLM labels Aug 24, 2026
…ceholder

Modifier.textPlaceholder now draws a hand-drawn squiggle seeded from the
placeholder pixel size instead of a solid pill. Two placeholders with the same
integer width and height draw the same squiggle, and any size difference draws a
different one. Adds a seeded path generator, determinism unit tests, and a
documentation screenshot.
@soulcramer
soulcramer force-pushed the feat/hand-drawn-text-placeholder branch from 81c375a to 629fa5a Compare August 24, 2026 22:17
@github-actions

Copy link
Copy Markdown
Contributor

🚨 UI regression detected! Checkout the paparazzi-delta artifact.
If these changes are expected, you can either:

  • manually run the gradlew cleanRecordPaparazziRelease and commit the new golden images
  • or ask @spark-ui-bot paparazzi golden images in this PR

@github-actions

This comment has been minimized.

@soulcramer
soulcramer marked this pull request as draft September 2, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component Related to a component enhancement New feature or request LLM Mainly built by a LLM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants