Skip to content

Conversation

@codegen-sh
Copy link
Contributor

@codegen-sh codegen-sh bot commented Sep 19, 2025

This PR implements a minimal HiddenField that registers with remix-hook-form without adding any extra DOM wrappers. This addresses layout issues when a hidden value is needed (e.g., lineItemId) and defaultValues already provide it.

What’s included:

  • New remix-hook-form/HiddenField that renders only a native <input type="hidden" /> while remaining fully registered with RHF.
  • Barrel export from remix-hook-form index.
  • Storybook story demonstrating usage with defaultValues and submission echo.

Requested by: Jake Ruesink

Notes:

  • Kept the component intentionally minimal per our form component patterns and to avoid unnecessary layout impact.
  • Story shows HiddenField alongside a visible TextField to verify no extra wrappers are introduced.

If you’d like me to add a direct value prop example (overriding defaultValues) or tests, let me know.


💻 View my workAbout Codegen
⛔ Remove Codegen from PR🚫 Ban action checks

…k docs\n\n- New HiddenField registers with remix-hook-form but renders only a native <input type="hidden"/> to avoid extra DOM\n- Export from remix-hook-form barrel\n- Add Storybook story demonstrating defaultValues submission and usage alongside visible fields
@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 19, 2025

📝 Storybook Preview: View Storybook

This preview will be updated automatically when you push new changes to this PR.

Note: The preview will be available after the workflow completes and the PR is approved for deployment.

… component)\n\nSummary: Adds new HiddenField that registers with remix-hook-form but renders only a native <input type="hidden"/> to avoid layout impact. Also exports from barrel and adds Storybook usage doc.
@jaruesink jaruesink merged commit ee9891f into main Sep 21, 2025
4 checks passed
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.

2 participants