Skip to content

Fix #609: move Form Edit textarea sizing styles to classes - #654

Open
arpitjain099 wants to merge 1 commit into
GSA-TTS:mainfrom
arpitjain099:chore/style-refactor-form-summary
Open

Fix #609: move Form Edit textarea sizing styles to classes#654
arpitjain099 wants to merge 1 commit into
GSA-TTS:mainfrom
arpitjain099:chore/style-refactor-form-summary

Conversation

@arpitjain099

Copy link
Copy Markdown

Fixes #609

I moved the textarea sizing styles out of inline style props and into class-based styling in the Form Edit components.

What changed:

  • added .nonResizableTextarea in formEditStyles.module.css
  • replaced inline textarea styles in:
    • FormSummaryEdit
    • AccordionRowPatternEdit
    • FieldsetEdit
    • SexPatternEdit
    • RepeaterPatternEdit
  • switched height and overflow to USWDS utility classes (height-15, overflow-auto)

Verification:

  • pnpm --filter @gsa-tts/forms-design lint -- src/FormManager/FormEdit/components/FormSummaryEdit.tsx src/FormManager/FormEdit/components/AccordionRowPatternEdit/AccordionRowPatternEdit.tsx src/FormManager/FormEdit/components/FieldsetEdit/FieldsetEdit.tsx src/FormManager/FormEdit/components/SexPatternEdit/SexPatternEdit.tsx src/FormManager/FormEdit/components/RepeaterPatternEdit/RepeaterPatternEdit.tsx

Note: local full workspace build is blocked in this environment by Node 26/native module install incompatibilities; lint on touched components passes.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@changeset-bot

changeset-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 25b1019

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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.

style refactor on Form Summary and other components

1 participant