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

[+] FO : Rich Snippets support, HTML5 markup, and ARIA roles #89

Closed
wants to merge 5 commits into from

Conversation

dunglas
Copy link
Contributor

@dunglas dunglas commented Dec 12, 2012

  • Adds some HTML5 semantic tags and Modernizr
  • Enables Google Rich Snippets support using Schema.org microdata
  • Adds main, navigation, search, complementary, contentinfo and banner ARIA roles

@dunglas
Copy link
Contributor Author

dunglas commented Dec 14, 2012

Added complementary and contentinfo ARIA roles.

@kpodemski
Copy link
Contributor

You can also remove "type" from javascript foreach and css

@dunglas
Copy link
Contributor Author

dunglas commented Dec 14, 2012

@kpodemski done.

@dunglas
Copy link
Contributor Author

dunglas commented Dec 14, 2012

An improvement will be to replace div of blocks in aside by article tags.
Maybe for a next version.

@kpodemski
Copy link
Contributor

Great, but now... maybe we should implement modernizr or simplier version of enabling HTML5 tags in IE? At this moment we have html5 tags but that's not work in crazIEst browser

@dunglas
Copy link
Contributor Author

dunglas commented Dec 14, 2012

Maybe html5shiv instead of the full Modernizr will do the job?

@kpodemski
Copy link
Contributor

yep, modernizr is too big

@gregoire-jianda
Copy link

Just htmlshiv and css classes would be great : http://modernizr.com/download/#-shiv-cssclasses

;)

@dunglas
Copy link
Contributor Author

dunglas commented Dec 14, 2012

@gbelorgey done

@dunglas
Copy link
Contributor Author

dunglas commented Jan 10, 2013

What about this PR?

@kpodemski
Copy link
Contributor

It's annoying that such positive changes are not taken into account!

@dunglas
Copy link
Contributor Author

dunglas commented Feb 5, 2013

@kpodemski I agree. I've released a standalone theme including those changes: http://dunglas.fr/2013/01/un-theme-prestashop-html5-optimise-pour-le-referencement-avec-rich-snippets/

@dunglas
Copy link
Contributor Author

dunglas commented Mar 16, 2013

Is there any plan to merge this PR? Anyone from the Prestashop team here?

@Jonadabe
Copy link
Contributor

Hi there,

Yes, that's a good question...

@kpodemski
Copy link
Contributor

@PrestaShop anyone?!

@ghost ghost assigned gRoussac Jul 23, 2013
@dunglas
Copy link
Contributor Author

dunglas commented Aug 28, 2013

@PrestaShop @rGaillard @vAugagneur @DamienMetzger @jeromenadaud can you merge this PR? Opened 9 months ago...

@LucasC
Copy link
Contributor

LucasC commented Oct 17, 2013

Hello @dunglas

I'm sorry that we didn't come back at you earlier. We are really interest in rich snippets - the default theme for the 1.6.0 version will include (some of) them. Your pull request is really great and it would be awesome if you could make this PR for the 1.6.0 theme (available in a few weeks). I personally guarantee that this PR would not stay open as long as this one.

Just one thing, please do not modify the charset :)

Thanks a lot for your involvement.

@dunglas
Copy link
Contributor Author

dunglas commented Oct 18, 2013

The charset is still UTF-8 but the new HTML5 syntax for meta is used.

@Ha99y
Copy link
Contributor

Ha99y commented Oct 18, 2013

prestashop 1.6 is going to be rich 💎
I ❤️ it

Than I won't have to edit after every upgrade

@gaillafr gaillafr closed this Dec 3, 2013
marionf pushed a commit to marionf/PrestaShop that referenced this pull request Oct 8, 2021
Use Prestashop requirements in Upgrade check list
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

10 participants