Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a text editor to render connection code when resuming connections #5449

Merged
merged 4 commits into from
Nov 22, 2024

Conversation

dfalbel
Copy link
Contributor

@dfalbel dfalbel commented Nov 22, 2024

Adresses #5219
Use a CodeEditorWidget to display the connections code so it's easily editable
and highlighted.

TODO:

  • Improve behavior of text editor before clicking edit - maybe make it unclickable and with a different type of cursor?

QA Notes

The resume connection modal should now show a text editor. After clicking in 'edit' the code should be editable and copy, resume should still work as expected.

Screen.Recording.2024-11-21.at.21.20.16.mov

@dfalbel dfalbel requested a review from softwarenerd November 22, 2024 00:24
@dfalbel dfalbel marked this pull request as ready for review November 22, 2024 00:24
softwarenerd
softwarenerd previously approved these changes Nov 22, 2024
Copy link
Contributor

@softwarenerd softwarenerd left a comment

Choose a reason for hiding this comment

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

LG! Comments were shared via Zoom.

@dfalbel dfalbel merged commit 042946e into main Nov 22, 2024
4 checks passed
@dfalbel dfalbel deleted the feature/resume-text-editor branch November 22, 2024 22:21
@github-actions github-actions bot locked and limited conversation to collaborators Nov 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants