Skip to content

fix(dashboard): correct overflow in a few settings edit forms - #11982

Merged
olivermrbl merged 8 commits into
medusajs:developfrom
leobenzol:fix/settings-forms-overflow
Jul 25, 2025
Merged

fix(dashboard): correct overflow in a few settings edit forms#11982
olivermrbl merged 8 commits into
medusajs:developfrom
leobenzol:fix/settings-forms-overflow

Conversation

@leobenzol

Copy link
Copy Markdown
Contributor

What

  • added overflow auto to the edit forms for shipping options, regions and store page

Why

  • currently these forms overflow for smaller resolutions(in my case a 1080p laptop at 150% native scaling). See screenshots below

Store page:

store edit form

Region page(looks ok but you cant scroll to the save button):
region edit form

Shipping option page:
so edit form

@changeset-bot

changeset-bot Bot commented Mar 25, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7c4a46d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 69 packages
Name Type
@medusajs/dashboard Patch
@medusajs/admin-bundler Patch
@medusajs/medusa Patch
@medusajs/test-utils Patch
@medusajs/medusa-oas-cli Patch
integration-tests-http Patch
@medusajs/analytics Patch
@medusajs/api-key Patch
@medusajs/auth Patch
@medusajs/cart Patch
@medusajs/currency Patch
@medusajs/customer Patch
@medusajs/file Patch
@medusajs/fulfillment Patch
@medusajs/index Patch
@medusajs/inventory Patch
@medusajs/link-modules Patch
@medusajs/locking Patch
@medusajs/notification Patch
@medusajs/order Patch
@medusajs/payment Patch
@medusajs/pricing Patch
@medusajs/product Patch
@medusajs/promotion Patch
@medusajs/region Patch
@medusajs/sales-channel Patch
@medusajs/stock-location Patch
@medusajs/store Patch
@medusajs/tax Patch
@medusajs/user Patch
@medusajs/workflow-engine-inmemory Patch
@medusajs/workflow-engine-redis Patch
@medusajs/oas-github-ci Patch
@medusajs/cache-inmemory Patch
@medusajs/cache-redis Patch
@medusajs/event-bus-local Patch
@medusajs/event-bus-redis Patch
@medusajs/analytics-local Patch
@medusajs/analytics-posthog Patch
@medusajs/auth-emailpass Patch
@medusajs/auth-github Patch
@medusajs/auth-google Patch
@medusajs/file-local Patch
@medusajs/file-s3 Patch
@medusajs/fulfillment-manual Patch
@medusajs/locking-postgres Patch
@medusajs/locking-redis Patch
@medusajs/notification-local Patch
@medusajs/notification-sendgrid Patch
@medusajs/payment-stripe Patch
@medusajs/core-flows Patch
@medusajs/framework Patch
@medusajs/js-sdk Patch
@medusajs/modules-sdk Patch
@medusajs/orchestration Patch
@medusajs/types Patch
@medusajs/utils Patch
@medusajs/workflows-sdk Patch
@medusajs/cli Patch
@medusajs/telemetry Patch
@medusajs/admin-sdk Patch
@medusajs/admin-shared Patch
@medusajs/admin-vite-plugin Patch
@medusajs/icons Patch
@medusajs/toolbox Patch
@medusajs/ui-preset Patch
create-medusa-app Patch
medusa-dev-cli Patch
@medusajs/ui Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Mar 25, 2025

Copy link
Copy Markdown

@steelrazor47 is attempting to deploy a commit to the medusajs Team on Vercel.

A member of the Team first needs to authorize it.

@olivermrbl

Copy link
Copy Markdown
Contributor

@fPolic, could you have a look at this one please?

@fPolic fPolic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@leobenzol

Copy link
Copy Markdown
Contributor Author

sorry to bother @olivermrbl, can this get merged? Wouldn't want it to slip by the next release :D

@leobenzol

Copy link
Copy Markdown
Contributor Author

should i add a changeset btw?

@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions Bot added the Stale label Jul 14, 2025
@github-actions github-actions Bot removed the Stale label Jul 15, 2025
@leobenzol

Copy link
Copy Markdown
Contributor Author

sorry @olivermrbl can we get this merged? It's been sitting for a while

@olivermrbl

Copy link
Copy Markdown
Contributor

@steelrazor47, will get it merged :)

@olivermrbl
olivermrbl merged commit 4da237f into medusajs:develop Jul 25, 2025
15 of 25 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.

4 participants