fix: switch to HTML text inputs with numeric validation

[jaredcwhite]

This PR addresses metrotranscom#658

• Addresses the issue in full
• Addresses only certain aspects of the issue

Description

Note: see comment below

The working theory is the issue with numbers changing randomly in the unit editing is due to input type="number" weirdness around mouse/trackpad scrolling. This PR switches to input type="text" and uses React Hook Form validation to verify integer input.

How Can This Be Tested/Reviewed?

Try editing units for a listing, income, or the miles radius on preferences.

Author Checklist:

• Added QA notes to the issue with applicable URLs
• Reviewed in a desktop view
• Reviewed in a mobile view
• Reviewed considering accessibility
• Added tests covering the changes
• Made corresponding changes to the documentation
• Ran yarn generate:client and/or created a migration when required

Review Process:

• Read and understand the issue
• Ensure the author has added QA notes
• Review the code itself from a style point of view
• Pull the changes down locally and test that the acceptance criteria is met
• Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

[netlify]

✅ Deploy Preview for bloom-exygy-dev ready!

Name	Link
🔨 Latest commit	68c56f0
🔍 Latest deploy log	https://app.netlify.com/sites/bloom-exygy-dev/deploys/667f0d83aee826000813b2a7
😎 Deploy Preview	https://deploy-preview-4128--bloom-exygy-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[jaredcwhite]

Cypress test failing seems to just be a glitch

[emilyjablonski]

What might min do here now that it's text?

[emilyjablonski]

After this change, I'm able to type letters into these numeric inputs, which I think we ideally want to avoid.

[netlify]

✅ Deploy Preview for partners-bloom-dev ready!

Name	Link
🔨 Latest commit	68c56f0
🔍 Latest deploy log	https://app.netlify.com/sites/partners-bloom-dev/deploys/667f0d834ae13700086188a9
😎 Deploy Preview	https://deploy-preview-4128--partners-bloom-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[jaredcwhite]

I've updated the PR with a combination of currency-specific fields and text fields with numeric validation. Only the currency fields expressively forbid typing letters, and they also have the byproduct of forcing decimal places (aka 500.00 rather than 500) which I'm concerned by cause trickle-down issues with display in other places. So…I'm not entirely sure we should consider this mergable without a broader convo.

[mcgarrye]

Is this PR still valid? If so, can we get it caught up to main and I can review Monday

[jaredcwhite]

@mcgarrye Because of the change where the decimal places are forced as mentioned above (typing 500 yields 500.00), I don't think we should merge this without discussing the possible ramifications.