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.

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

Fix HTML5 validation error #38

Merged
merged 2 commits into from
Feb 1, 2016
Merged

Fix HTML5 validation error #38

merged 2 commits into from
Feb 1, 2016

Conversation

CGarces
Copy link
Contributor

@CGarces CGarces commented Jan 30, 2016

Removed itemprop from canonical url due html validation error

See it in this sample page and the validation

The url is already send in the ld+json data when seo_site_title is not null. Usually seo_site_title always will have a value assigned,

@CGarces
Fix HTML5 validation error
56c7f4e 
Removed itemprop from canonical url due html validation error
@CGarces
Fix travis-ci errors
eac9501
@pathawks
Copy link
Member

👍

@benbalter
Copy link
Collaborator

Thanks @CGarces!

benbalter added a commit that referenced this pull request Feb 1, 2016
@benbalter
Merge pull request #38 from CGarces/patch-1
aca4b60 
Fix HTML5 validation error
@benbalter benbalter merged commit aca4b60 into jekyll:master Feb 1, 2016
@CGarces CGarces deleted the patch-1 branch February 1, 2016 15:56
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
frozen-due-to-age
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@CGarces @pathawks @benbalter @jekyllbot