Skip to content

FE-1131: petrinaut use text area#8947

Merged
alex-e-leon merged 2 commits into
mainfrom
FE-1131-petrinaut-use-text-area
Jul 3, 2026
Merged

FE-1131: petrinaut use text area#8947
alex-e-leon merged 2 commits into
mainfrom
FE-1131-petrinaut-use-text-area

Conversation

@alex-e-leon

Copy link
Copy Markdown
Contributor

🌟 What is the purpose of this PR?

Adds TextArea to petrinaut.

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

@alex-e-leon alex-e-leon requested a review from kube July 3, 2026 13:09
@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Ready Ready Preview, Comment Jul 3, 2026 1:28pm
hashdotdesign-tokens Ready Ready Preview, Comment Jul 3, 2026 1:28pm
petrinaut Ready Ready Preview, Comment Jul 3, 2026 1:28pm

@cursor

cursor Bot commented Jul 3, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
UI-only form and component styling changes with no auth, data, or API impact; TextArea default resize and className placement may slightly affect other consumers.

Overview
Petrinaut metric and scenario description fields now use the design-system TextArea instead of locally styled native <textarea> elements, with shared sm sizing and an 80px minimum height. Duplicate textarea styles are removed, and the metric name field’s TextInput is aligned to sm.

TextArea in ds-components gets small layout fixes: a calculated minHeight on the root, tighter subtle overlay anchoring (no extra pixel offsets), transparent subtle root background, and several token overrides moved to wrapper for disabled/subtle sizing. className applies to the inner box (root) rather than the outer wrapper; the default resize prop is vertical instead of both.

Reviewed by Cursor Bugbot for commit ba32e58. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions github-actions Bot added area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team area/apps > hash.design Affects the `hash.design` design site (app) labels Jul 3, 2026
@alex-e-leon alex-e-leon force-pushed the FE-1131-petrinaut-use-text-area branch from 2cc65c4 to 7558cd3 Compare July 3, 2026 13:09

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7558cd3. Configure here.

kube
kube previously approved these changes Jul 3, 2026

@kube kube left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM
Always cool to see in-component CSS classes disappear.

@alex-e-leon alex-e-leon added this pull request to the merge queue Jul 3, 2026
Merged via the queue into main with commit 253c32d Jul 3, 2026
48 checks passed
@alex-e-leon alex-e-leon deleted the FE-1131-petrinaut-use-text-area branch July 3, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/apps > hash.design Affects the `hash.design` design site (app) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

2 participants