Skip to content

Update open-close design doc to accept the hash & slash alternative #535

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

Merged
merged 2 commits into from
Nov 28, 2023

Conversation

eemeli
Copy link
Collaborator

@eemeli eemeli commented Nov 27, 2023

This reflects the consensus reached during the 2023-11-20 call, to accept {#foo}...{/foo} as the open-close syntax.

The previously proposed {#foo/} syntax for standalone elements did not reach consensus, and so is left out here.

Examples are updated to use the new syntax, and the {+foo}...{-foo} alternative is documented as such.

@eemeli eemeli requested review from aphillips and stasm November 27, 2023 16:00
@eemeli eemeli added design Design document or issues related to design Agenda+ Requested for upcoming teleconference labels Nov 27, 2023
@aphillips aphillips merged commit 87331f9 into main Nov 28, 2023
@aphillips aphillips deleted the hash-n-slash branch November 28, 2023 20:04
@stasm
Copy link
Collaborator

stasm commented Nov 29, 2023

During the teleconference on Monday I supported merging this PR as proposed. However, I had forgotten about this part:

The previously proposed {#foo/} syntax for standalone elements did not reach consensus, and so is left out here.

I think we should phrase this differently. Right now, the doc recognizes that standalone elements are still an open question, but the documented proposal goes one step further: it suggests to use the registry to encode the standalone aspect. I filed #542 with my suggestion for how to improve the doc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+ Requested for upcoming teleconference design Design document or issues related to design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants