Skip to content

Improve Export Configurations modal with raw and human-readable views#1423

Open
SukuWc wants to merge 3 commits intostablefrom
SUKU-raw-code
Open

Improve Export Configurations modal with raw and human-readable views#1423
SukuWc wants to merge 3 commits intostablefrom
SUKU-raw-code

Conversation

@SukuWc
Copy link
Member

@SukuWc SukuWc commented Feb 23, 2026

Summary

  • Display both raw code and human-readable (expanded) Lua side by side in the Export Configurations modal
  • Replace legacy execCommand clipboard hack with navigator.clipboard.writeText
  • Migrate ProfileCloud config link copy to navigator.clipboard and remove the now-unused window.electron.clipboard IPC path

Test plan

  • Open Export Configurations modal and verify both "Raw code" and "Human readable" panels appear side by side
  • Verify Copy button on each panel copies the correct content with newlines preserved
  • Verify Profile Cloud "copy config link" still works
  • Confirm no regressions in other clipboard-related features

🤖 Generated with Claude Code

- Display both raw code and human-readable (expanded) Lua side by side
- Replace legacy execCommand clipboard hack with navigator.clipboard.writeText
- Migrate ProfileCloud config link copy to navigator.clipboard as well
- Remove now-unused window.electron.clipboard IPC path from main/preload

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Feb 23, 2026

Visit the preview URL for this PR (updated for commit 84cb2f6):

https://grid-editor-web--pr1423-suku-raw-code-091n2otx.web.app

(expires Wed, 25 Mar 2026 16:31:46 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2b65ba6ef19c55d367eaffd04e46bcde25305d6f

@github-actions
Copy link

github-actions bot commented Feb 23, 2026

Visit the preview URL for this PR (updated for commit 84cb2f6):

https://grid-playwright-result--pr1423-suku-raw-code-2toml38v.web.app

(expires Thu, 26 Feb 2026 16:39:58 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 0f764ceb264072188bf1a5dee10613d8f10b4ca2

@SukuWc SukuWc moved this to Ready for QA in Editor Roadmap Mar 4, 2026
@Greg-Orca Greg-Orca moved this from Ready for QA to QA in progress in Editor Roadmap Mar 4, 2026
@Greg-Orca
Copy link
Contributor

passed

@Greg-Orca Greg-Orca moved this from QA in progress to Done in Editor Roadmap Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants