Skip to content

Framework react test#6

Merged
natemoo-re merged 4 commits intomainfrom
framework-react-test
Aug 31, 2021
Merged

Framework react test#6
natemoo-re merged 4 commits intomainfrom
framework-react-test

Conversation

@drwpow
Copy link
Copy Markdown
Member

@drwpow drwpow commented Aug 27, 2021

OK I lied; I actually did poke at it a little more when I said I wasn’t gonna. So adding the example from our examples/framework-react code, it actually does hang in the parser so I guess that makes more sense now!

want{
name: "basic (no frontmatter)",
source: `<button>Click</button>`,
want: want{
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I was just messing around adding these. Dunno if I like em or not

<body>
<main>
<Counter {...someProps} client:visible>
<h1>Hello React!</h1>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This slot, along with the spread props, may be what’s hanging it up? Anyway this test just never completes, which matches what I was seeing from the npm version. Let’s take a look at it next week!

@drwpow drwpow force-pushed the framework-react-test branch 4 times, most recently from dabc350 to 06d3e27 Compare August 31, 2021 18:31
@natemoo-re natemoo-re force-pushed the framework-react-test branch from e8cbabb to 5073605 Compare August 31, 2021 21:20
@natemoo-re natemoo-re merged commit 2c8e445 into main Aug 31, 2021
@natemoo-re natemoo-re deleted the framework-react-test branch August 31, 2021 21:32
Princesseuh pushed a commit that referenced this pull request Feb 20, 2026
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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