fix: overflow in hyperparameter modal

[keita-determined]

Ticket

ET-308

Description

Screen.Recording.2024-07-09.at.5.01.41.PM.mov

Test Plan

• open hyperaparatmeter search in experiment detail page
• Check if the vavlue table does not overflow when the page is narrow and in mobile view

Checklist

• Changes have been manually QA'd
• New features have been approved by the corresponding PM
• User-facing API changes have the "User-facing API Change" label
• Release notes have been added as a separate file under docs/release-notes/
  See Release Note for details.
• Licenses have been included for new code which was copied and/or modified from any external code

[netlify]

✅ Deploy Preview for determined-ui ready!

Name	Link
🔨 Latest commit	996f4e7
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/668f0544e2f28500081481c6
😎 Deploy Preview	https://deploy-preview-9626--determined-ui.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.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.79%. Comparing base (3a8c042) to head (996f4e7).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9626      +/-   ##
==========================================
- Coverage   52.91%   50.79%   -2.13%     
==========================================
  Files        1255      438     -817     
  Lines      153215    74057   -79158     
  Branches     3245     3245              
==========================================
- Hits        81081    37616   -43465     
+ Misses      71983    36290   -35693     
  Partials      151      151              

Flag	Coverage Δ
harness	?
web	51.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 817 files with indirect coverage changes

[johnkim-det]

this does address the issue, but I noticed something when trying out the netlify preview on my iPhone -- looks like safari/webkit makes the rows have an additional vertical space that looks broken. but this doesn't happen on latest-main on my iPhone. can you take a look?

[keita-determined]

this does address the issue, but I noticed something when trying out the netlify preview on my iPhone -- looks like safari/webkit makes the rows have an additional vertical space that looks broken. but this doesn't happen on latest-main on my iPhone. can you take a look?

@johnkim-det good catch. fixed the issue and tested on chrome, firefox, and safari.
i wish there was a better way to test things on all browsers instead of the manual testing,

[johnkim-det]

LGTM