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

Change examples of <meta ... href=""> to <link ... href=""> #478

Closed
wants to merge 1 commit into
base: sdo-gozer
from

Conversation

Projects
None yet
3 participants
@dbs
Contributor

dbs commented May 7, 2015

A few instances of had crept into examples, but

is the preferred formulation.

Related to #77, which focuses on the more worrisome usage in examples.

Signed-off-by: Dan Scott dan@coffeecode.net

Change examples of <meta ... href=""> to <link ... href="">
A few instances of <meta property="" href=""> had crept into examples, but
<link property="" href=""> is the preferred formulation.

Related to #77, which focuses on the more worrisome <meta property=""
content="http://example.com"> usage in examples.

Signed-off-by: Dan Scott <dan@coffeecode.net>
@danbri

This comment has been minimized.

Show comment
Hide comment
@danbri

danbri May 12, 2015

Contributor

Punting this to next release as I am scared of corner-cases in our pseudo-parser.

Contributor

danbri commented May 12, 2015

Punting this to next release as I am scared of corner-cases in our pseudo-parser.

@danbri danbri added this to the 2015 Q2 milestone May 12, 2015

@danbri danbri self-assigned this May 12, 2015

@danbri

This comment has been minimized.

Show comment
Hide comment
@danbri

danbri May 12, 2015

Contributor

And btw, thanks for taking the time! Sorry not to have got it in this release...

Contributor

danbri commented May 12, 2015

And btw, thanks for taking the time! Sorry not to have got it in this release...

RichardWallis added a commit that referenced this pull request Sep 21, 2015

RichardWallis added a commit that referenced this pull request Sep 21, 2015

@RichardWallis

This comment has been minimized.

Show comment
Hide comment
@RichardWallis

RichardWallis Sep 21, 2015

Contributor

merged into #787

Thanks @dbs

Contributor

RichardWallis commented Sep 21, 2015

merged into #787

Thanks @dbs

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