Skip to content

Comments

fix(shared): save gear modal on Enter submit#5534

Merged
idoshamun merged 3 commits intomainfrom
eng-761-feedback-general-feedback-return-key-closes-input-modal
Feb 21, 2026
Merged

fix(shared): save gear modal on Enter submit#5534
idoshamun merged 3 commits intomainfrom
eng-761-feedback-general-feedback-return-key-closes-input-modal

Conversation

@idoshamun
Copy link
Member

@idoshamun idoshamun commented Feb 21, 2026

Summary

  • fix GearModal form field wiring so react-hook-form receives input updates when typing
  • ensure Enter/Return and clicking Add gear follow the same submit path and persist gear before closing
  • add a focused regression test for both submit actions

Key decisions

  • kept the fix minimal and local to GearModal by chaining register('name').onChange instead of introducing a controller refactor
  • removed unnecessary key-specific logic and relied on native form submit behavior for Enter

Closes ENG-761


Created by Huginn 🐦‍⬛

Preview domain

https://eng-761-feedback-general-feedbac.preview.app.daily.dev

@vercel
Copy link

vercel bot commented Feb 21, 2026

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Feb 21, 2026 6:32am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Feb 21, 2026 6:32am

Request Review

@idoshamun idoshamun merged commit bada13e into main Feb 21, 2026
11 checks passed
@idoshamun idoshamun deleted the eng-761-feedback-general-feedback-return-key-closes-input-modal branch February 21, 2026 06:37
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