Skip to content

Conversation

@dan-rukas
Copy link
Member

@dan-rukas dan-rukas commented Jan 20, 2026

Context

This PR improves accessibility and simplifies layout styles in the UserPreferencesModal settings, building on the i18n work in #5561.

Changes & Results

  • Reduced column gap - gap-x-16gap-x-10
  • Restored alignment so labels align vertically with their input fields - items-startitems-center
  • Simplified label classes to just text-base. The additional flex and whitespace utilities were explicitly controlling text wrapping, but this happens naturally with default browser behavior. The adjusted column gap provides more space for longer translations.

Testing

Visually tested with multiple localizations to ensure labels display correctly with proper wrapping overflow.

Checklist

PR

  • My Pull Request title is descriptive, accurate and follows the
    semantic-release format and guidelines.

Code

  • My code has been well-documented (function documentation, inline comments,
    etc.)

Public Documentation Updates

  • [] The documentation page has been updated as necessary for any public API
    additions or removals.

Tested Environment

  • OS: mac)S Sequoia 15.7.3 (24G419), Apple M4 Pro
  • Node version: 20.19.1
  • Browser: 143.0.7499.193 (Official Build) (arm64)

@netlify
Copy link

netlify bot commented Jan 20, 2026

Deploy Preview for ohif-dev ready!

Name Link
🔨 Latest commit 107c224
🔍 Latest deploy log https://app.netlify.com/projects/ohif-dev/deploys/696f972d58288300085822e2
😎 Deploy Preview https://deploy-preview-5730--ohif-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@jbocce jbocce self-requested a review January 20, 2026 15:06
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