Skip to content

[backport cloud/1.37] Updates: More Modal Modification#8308

Merged
DrJKL merged 1 commit intocloud/1.37from
backport-8256-to-cloud-1.37
Jan 25, 2026
Merged

[backport cloud/1.37] Updates: More Modal Modification#8308
DrJKL merged 1 commit intocloud/1.37from
backport-8256-to-cloud-1.37

Conversation

@DrJKL
Copy link
Contributor

@DrJKL DrJKL commented Jan 25, 2026

Backport of #8256 to cloud/1.37

┆Issue is synchronized with this Notion page by Unito

Refactors modal dialog layouts for improved flexibility and consistency.

**Changes:**
- Add dedicated slot for left panel header title with dynamic
content/icons
- Consolidate side panel rendering within `BaseModalLayout`
- Remove redundant `PanelHeader` and `RightSidePanel` components
- Apply `select-none` to text elements to prevent accidental selection

---------

Co-authored-by: Amp <amp@ampcode.com>
@DrJKL DrJKL added the backport Backporting a PR onto a release candidate label Jan 25, 2026
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jan 25, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 25, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • backport

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 25, 2026

🎭 Playwright Tests: ⚠️ Passed with flaky tests

Results: 502 passed, 0 failed, 2 flaky, 8 skipped (Total: 512)

❌ Failed Tests

📊 Browser Reports
  • chromium: View Report (✅ 492 / ❌ 0 / ⚠️ 1 / ⏭️ 8)
  • chromium-2x: View Report (✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • chromium-0.5x: View Report (✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • mobile-chrome: View Report (✅ 7 / ❌ 0 / ⚠️ 1 / ⏭️ 0)

@github-actions
Copy link

github-actions bot commented Jan 25, 2026

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 01/25/2026, 12:54:22 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@DrJKL DrJKL merged commit ec91aa8 into cloud/1.37 Jan 25, 2026
33 checks passed
@DrJKL DrJKL deleted the backport-8256-to-cloud-1.37 branch January 25, 2026 01:50
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:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant