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

Define CRS/CRD abbreviations, fixes #636 #654

Merged
merged 1 commit into from
Oct 19, 2022
Merged

Define CRS/CRD abbreviations, fixes #636 #654

merged 1 commit into from
Oct 19, 2022

Conversation

fantasai
Copy link
Collaborator

@fantasai fantasai commented Sep 30, 2022

index.bs Outdated Show resolved Hide resolved
@TallTed
Copy link
Member

TallTed commented Oct 1, 2022

@fantasai — Looking at the SVG changes, I noticed that e.g. instances are not surrounded by commas, but should be according to the external style guides to which the W3 style guide points. Is this sufficient call to your attention, or would you like new issue(s) and/or PR(s)?

@fantasai
Copy link
Collaborator Author

@TallTed It's an image, not a paragraph, so actually I think it's correct to not have commas. Punctuation in images is often minimized.

@TallTed
Copy link
Member

TallTed commented Oct 13, 2022

@fantasai

@TallTed It's an image, not a paragraph, so actually I think it's correct to not have commas. Punctuation in images is often minimized.

At best, that's a slippery slope kind of argument. Where does such minimization end?

I have found no style guidance saying anything like, "it's OK to use less punctuation when text is shown within an image, and here's how much less is OK."

@fantasai fantasai merged commit 55a2e15 into w3c:main Oct 19, 2022
@fantasai fantasai deleted the crs branch October 19, 2022 15:50
@frivoal frivoal added the Closed: Accepted The issue has been addressed, though not necessarily based on the initial suggestion label Mar 2, 2023
@frivoal frivoal added this to the Process 2023 milestone Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed: Accepted The issue has been addressed, though not necessarily based on the initial suggestion Type: Editorial improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants