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

hds-code-editor - Attach EditorView instance to modified element #2779

Merged
merged 4 commits into from
Mar 27, 2025

Conversation

zamoore
Copy link
Contributor

@zamoore zamoore commented Mar 19, 2025

๐Ÿ“Œ Summary

If merged, this PR will attach the CodeMirror EditorView instance to the element modified by the hds-code-editor modifier.

๐Ÿ› ๏ธ Detailed description

Why?

Needed for supporting editor interactions in the acceptance tests of our downstream consumers

How?

When the editor is instantiated by the modifier, we attach the instance to the modified DOM Element.

๐Ÿ”— External links

Jira ticket: HDS-4677


๐Ÿ‘€ Component checklist

๐Ÿ’ฌ Please consider using conventional comments when reviewing this PR.

Copy link

vercel bot commented Mar 19, 2025

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Updated (UTC)
hds-showcase โœ… Ready (Inspect) Visit Preview Mar 26, 2025 9:44pm
hds-website โœ… Ready (Inspect) Visit Preview Mar 26, 2025 9:44pm

@zamoore zamoore changed the title Zamoore/hds 4677/attach editor instance to dom hds-code-editor - Attach EditorView instance to modified element Mar 19, 2025
@zamoore zamoore force-pushed the zamoore/HDS-4677/attach-editor-instance-to-dom branch from 6cd290f to 897cc16 Compare March 20, 2025 21:04
@zamoore zamoore marked this pull request as ready for review March 20, 2025 22:05
@zamoore zamoore requested a review from a team as a code owner March 20, 2025 22:05
alex-ju
alex-ju previously approved these changes Mar 25, 2025
Copy link
Member

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

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

Left a suggestion for your assessment, otherwise looks good

@zamoore zamoore merged commit f9b4b08 into main Mar 27, 2025
16 checks passed
@zamoore zamoore deleted the zamoore/HDS-4677/attach-editor-instance-to-dom branch March 27, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants