Skip to content

Conversation

Josh-Cena
Copy link
Member

@Josh-Cena Josh-Cena commented Jul 23, 2025

Part of mdn/content#16120. This example is interesting but too long, and uses bootstrap, so it's better to be standalone.

cc @bsmth

@Josh-Cena Josh-Cena requested a review from a team as a code owner July 23, 2025 04:09
@Josh-Cena Josh-Cena requested review from pepelsbey and removed request for a team July 23, 2025 04:09
@bsmth bsmth self-requested a review July 23, 2025 09:13
@bsmth
Copy link
Member

bsmth commented Jul 23, 2025

There was a couple of things here to get it working, I committed them here: https://github.com/Josh-Cena/mdn-dom-examples/compare/execcommand-ex...bsmth:dom-examples:execcommand-ex-brian?expand=1 - the diff is a little messed up though because of line endings, I ran:

git add --renormalize .

@Josh-Cena
Copy link
Member Author

Thanks! I keep messing up the line endings because I'm not in the habit yet and I'm soon going back to mac anyway.

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Great, thanks 👍🏻

@bsmth bsmth merged commit 8c3d2a9 into mdn:main Jul 24, 2025
1 check passed
@Josh-Cena Josh-Cena deleted the execcommand-ex branch July 24, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants