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

docs(webc): Formatting tweaks #1584

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

docs(webc): Formatting tweaks #1584

wants to merge 2 commits into from

Conversation

Zearin
Copy link
Contributor

@Zearin Zearin commented May 14, 2023

Mainly wanted to add a little clarity to some code examples that did not specify whether the input was from a page or a component.

I found myself reading and re-reading these docs because I wasn’t sure if certain webc:* attributes only applied in component files or page files.

Some examples also included the input code and output code in the same block. That was understandable, but it’s also inconsistent with the way most other examples are shown throughout the docs. I tried to bring these cases more in line with typical usage.

I also noticed one paragraph that should probably be a callout, so I added the codes to make it so.

Finally, there are a few commas and other small copy-edits to the text itself.

@Zearin
Copy link
Contributor Author

Zearin commented May 19, 2023

@zachleat ping

@Zearin
Copy link
Contributor Author

Zearin commented Oct 18, 2023

@zachleat ping (6 mo; easy merge 🤓)

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.

None yet

1 participant