Skip to content

Conversation

webfiltered
Copy link
Contributor

@webfiltered webfiltered commented Sep 18, 2025

Summary

Cherry-picked PR #5605 (Add desktop dialogs framework) to core/1.27 branch for hotfix release.

Cherry-picked commits

Changes

  • Data-driven dialog structure in desktopDialogs.ts
  • Dynamic dialog view component with i18n support
  • Button action types: openUrl, close, cancel
  • Button severity levels for styling (primary, secondary, danger, warn)
  • Fallback invalid dialog for error handling
  • i18n collection script updated for dialog strings

Testing

  • Typecheck passed ✓
  • Cherry-pick applied cleanly without conflicts

Impact

This adds the desktop dialog framework feature to the stable 1.27 branch, allowing desktop applications to display standardized dialogs.

┆Issue is synchronized with this Notion page by Unito

### Summary

Adds desktop dialog framework with data-driven dialog definitions.

### Changes

- Data-driven dialog structure in `desktopDialogs.ts`
- Dynamic dialog view component with i18n support
- Button action types: openUrl, close, cancel
- Button severity levels for styling (primary, secondary, danger, warn)
- Fallback invalid dialog for error handling
- i18n collection script updated for dialog strings
@webfiltered webfiltered requested a review from a team as a code owner September 18, 2025 04:45
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Sep 18, 2025
Copy link

github-actions bot commented Sep 18, 2025

🎭 Playwright Test Results

All tests passed!

⏰ Completed at: 09/18/2025, 04:57:29 AM UTC

📈 Summary

  • Total Tests: 450
  • Passed: 421 ✅
  • Failed: 0
  • Flaky: 0
  • Skipped: 29 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 414 / ❌ 0 / ⚠️ 0 / ⏭️ 29
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 4 / ❌ 0 / ⚠️ 0 / ⏭️ 0

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

@webfiltered webfiltered merged commit c39a03f into core/1.27 Sep 18, 2025
22 checks passed
@webfiltered webfiltered deleted the hotfix/1.27.6-20250918-144358 branch September 18, 2025 06:26
@AustinMroz AustinMroz mentioned this pull request Sep 30, 2025
AustinMroz added a commit that referenced this pull request Sep 30, 2025
## What's Changed
### 🚀 Features
- add pricing for new api nodes (#5725)
- Rework desktop install / startup UX (#5292)

### 🐛 Bug Fixes
- Make experiment asset api setting hidden (#5861)
- Only add the listeners for DomWidget components once (#5849)
- fix invalid JSON (contains merge conflict markers) in 1.27 RC branch's
locale json (#5839)
- fix: Status indicator and close button appearing together  (#5741)
- Fix cyclic prototype errors with subgraphNodes (#5650)
- fix: don't immediately close missing nodes dialog if manager is
disabled (#5649)
- Fix SaveAs (#5644)

### 🔧 Maintenance  
- Add JSON validation CI workflow (#5838)
- Cherry-pick manager flag refactor to core/1.27 (#5646)
- Cherry-pick desktop dialogs framework to core/1.27 (#5634)

**Full Changelog**:
v1.27.5...v1.27.7

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5862-1-27-7-27e6d73d3650819aa712e682f6b88077)
by [Unito](https://www.unito.io)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants