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.

Sign up for GitHub

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

Jump to bottom

Simplify symbol examples #207

Merged

snidersd merged 6 commits into main from simplify-symbol-examples

May 16, 2022

Contributor

matatk commented May 6, 2022 •

edited

Loading

As discussed we're simplifying the symbol examples to remove anything that may be construed as translation. This PR also tightens up the parsing rules a bit, and doesn't stop us from having multiple symbols within one attribute value in future.

HTML Previews:

Before: https://w3c.github.io/personalization-semantics/content/#symbol-explanation
After: http://raw.githack.com/w3c/personalization-semantics/simplify-symbol-examples/content/index.html#symbol-explanation (also the examples section, below)

Questions:

I have some questions about this; the editorial question is: have I marked up "the plus sign (+)" part inkeeping with W3C house style?

For the other questions about this, I've started a thread on the mailing list: https://lists.w3.org/Archives/Public/public-personalization-tf/2022May/0005.html

This may clarify some matters relating to #203.

matatk added 6 commits

May 6, 2022 13:35


          Remove trailing space

53cee34


          Fix example indentation

92fb9ec


          Remove requirement for spaces around the plus sign

efad34c


          Simplify examples

8aa00f1


          Fix spacing within example code

2f472d2


          Typos

b821f98

matatk force-pushed the simplify-symbol-examples branch from 9bfc4c2 to b821f98 Compare

May 6, 2022 17:29

snidersd merged commit 952e582 into main

github-actions bot added a commit that referenced this pull request


          Simplify symbol examples (#207)

SHA: 952e582
Reason: push, by @snidersd

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

github-actions bot added a commit that referenced this pull request


          Simplify symbol examples (#207)

d5f5681

SHA: 952e582
Reason: push, by @snidersd

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

github-actions bot added a commit that referenced this pull request


          Simplify symbol examples (#207)

SHA: 952e582
Reason: push, by @snidersd

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

github-actions bot added a commit that referenced this pull request


          Simplify symbol examples (#207)

4be5ed8

SHA: 952e582
Reason: push, by @snidersd

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

github-actions bot added a commit that referenced this pull request


          Simplify symbol examples (#207)

41bbf77

SHA: 952e582
Reason: push, by @snidersd

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

matatk deleted the simplify-symbol-examples branch

June 27, 2022 14:20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment