Skip to content

[backport cloud/1.36] fix: Button sizing in modals and asset browser#7921

Merged
DrJKL merged 1 commit intocloud/1.36from
backport-7920-to-cloud-1.36
Jan 9, 2026
Merged

[backport cloud/1.36] fix: Button sizing in modals and asset browser#7921
DrJKL merged 1 commit intocloud/1.36from
backport-7920-to-cloud-1.36

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Jan 9, 2026

Backport of #7920 to cloud/1.36

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

## Summary

Fix button sizing inconsistencies in modal dialogs and the asset
browser.

## Changes

- **What**: Fix Import button using responsive size (`lg`/`icon` based
on breakpoint) and ensure Modal Close button has explicit `w-10` width
for consistent sizing.

## Review Focus

Button sizing consistency across the modal UI.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-7920-fix-Button-sizing-in-modals-and-asset-browser-2e36d73d365081fc997af8be1e928049)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Jan 9, 2026
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner January 9, 2026 03:03
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Jan 9, 2026
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 9, 2026
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 01/09/2026, 03:05:14 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@github-actions
Copy link

github-actions bot commented Jan 9, 2026

🎭 Playwright Test Results

⚠️ Tests passed with flaky tests

⏰ Completed at: 01/09/2026, 03:13:53 AM UTC

📈 Summary

  • Total Tests: 512
  • Passed: 502 ✅
  • Failed: 0
  • Flaky: 2 ⚠️
  • Skipped: 8 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 490 / ❌ 0 / ⚠️ 2 / ⏭️ 8
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 9 / ❌ 0 / ⚠️ 0 / ⏭️ 0

🎉 Click on the links above to view detailed test results for each browser configuration.

@DrJKL DrJKL merged commit be8ee3d into cloud/1.36 Jan 9, 2026
37 checks passed
@DrJKL DrJKL deleted the backport-7920-to-cloud-1.36 branch January 9, 2026 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants