Skip to content

key and ref for all JSX elements#2925

Closed
kraftwer1 wants to merge 1 commit intostenciljs:masterfrom
kraftwer1:patch-3
Closed

key and ref for all JSX elements#2925
kraftwer1 wants to merge 1 commit intostenciljs:masterfrom
kraftwer1:patch-3

Conversation

@kraftwer1
Copy link
Copy Markdown
Contributor

Every element that is generated by JSX can have a key or a ref attribute.
This allows, for example: <slot ref={(el) => this.slotEl = el} />

Every element that is generated by JSX can have a key or a ref attribute.
This allows, for example: `<slot ref={(el) => this.slotEl = el />`
@kraftwer1
Copy link
Copy Markdown
Contributor Author

@rwaskiewicz any comments on why you've closed this? Thanks.

@rwaskiewicz
Copy link
Copy Markdown
Member

@kraftwer1 I'm not sure what happened here. Around that time I moved Stencil's primary branch from using 'master' to 'main' naming, and it looks like the issue was closed on the same/date time 🤔 My understanding was that GitHub would not be closing open PRs. So I'd guess one of two things happened here:

  1. there was an issue when i did the renaming on GH's end and that closed the issue
  2. I manually deleted master and that closed the PR.

Either way this shouldn't have been closed. Let me reopen it and make sure nothing else got closed on that date. Sorry about that!

@rwaskiewicz
Copy link
Copy Markdown
Member

@kraftwer1 odd...for some reason only this PR was closed when #3096 was merged (to make the switch from master->main). I don't seem to have that ability to re-open this (I suspect because master is no longer available). Would you be amenable to re-opening a new PR against main for me? Again, so sorry about that!

@kraftwer1
Copy link
Copy Markdown
Contributor Author

Hi @rwaskiewicz thanks for your quick response. No problem, here's the new one:
#3177

@kraftwer1 kraftwer1 mentioned this pull request Dec 15, 2021
14 tasks
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