fix: un markup microdata sur les prix qui est valide et fonctionnel et ne déclenche pas SafeHtml #12

Merged
cerdic merged 1 commits from issue_6 into master 2 months ago
cerdic commented 2 months ago
Owner

On ajoute donc une référence à la spec microdata et on déplace les itemprop+content sur les span, ce qui fait aussi bien le taf et donne un résultat correct dans https://validator.schema.org/

Refs: #6

Le markup généré est maintenant de la forme

<span itemscope="itemscope" itemtype="https://schema.org/PriceSpecification"><span class="montant" data-montant-nombre="30" data-montant-devise="EUR" itemprop="price" content="30">30,00 <span class="montant__devise" itemprop="priceCurrency" content="EUR">EUR</span></span></span>

ce qui ne doit rien casser pour les affichages, et passe bien dans SafeHtml jusqu'à SPIP 4.1.x

En SPIP 4.2 basé sur la lib HtmlPurifier on perd les attributs microdata, ce qui doit être corrigé cf spip/safehtml#4790

- le markup microdata n'était pas reconnu selon https://validator.schema.org/ car ne référençant par la https://schema.org/PriceSpecification (donc totalement inutile) - le markup utilisait des balises `<meta>` qui triggent safehtml() et donc déclenchent des alertes pour rien On ajoute donc une référence à la spec microdata et on déplace les itemprop+content sur les span, ce qui fait aussi bien le taf et donne un résultat correct dans https://validator.schema.org/ Refs: #6 Le markup généré est maintenant de la forme ```html <span itemscope="itemscope" itemtype="https://schema.org/PriceSpecification"><span class="montant" data-montant-nombre="30" data-montant-devise="EUR" itemprop="price" content="30">30,00 <span class="montant__devise" itemprop="priceCurrency" content="EUR">EUR</span></span></span> ``` ce qui ne doit rien casser pour les affichages, et passe bien dans SafeHtml jusqu'à SPIP 4.1.x En SPIP 4.2 basé sur la lib HtmlPurifier on perd les attributs microdata, ce qui doit être corrigé cf https://git.spip.net/spip/safehtml/issues/4790
cerdic added 1 commit 2 months ago
b4e488a2bc fix: un markup microdata sur les prix qui est valide et fonctionnel et ne déclenche pas SafeHtml
tcharlss requested changes 2 months ago
tcharlss left a comment
Owner

Ok mais par contre il faut garder un <span class="montant"> à la racine, pas de span fantôme sans classe impossible à cibler en CSS.

Au final quasi la même chose (sans saut de ligne, là c'est juste pour la lisibilité) :

<span class="montant" data-montant-nombre="100" data-montant-devise="EUR" itemscope="itemscope" itemtype="https://schema.org/PriceSpecification">
	<span class="montant__inner" itemprop="price" content="100">
		100,00&nbsp;<span class="montant__devise" itemprop="priceCurrency" content="EUR">EUR</span>
	</span>
</span>
Ok mais par contre il faut garder un `<span class="montant"`> à la racine, pas de span fantôme sans classe impossible à cibler en CSS. Au final quasi la même chose (sans saut de ligne, là c'est juste pour la lisibilité) : ```html <span class="montant" data-montant-nombre="100" data-montant-devise="EUR" itemscope="itemscope" itemtype="https://schema.org/PriceSpecification"> <span class="montant__inner" itemprop="price" content="100"> 100,00&nbsp;<span class="montant__devise" itemprop="priceCurrency" content="EUR">EUR</span> </span> </span> ```
cerdic added 1 commit 2 months ago
Poster
Owner

j'ai complété en ce sens et vérifié qu'on valide toujours le shema.org

j'ai complété en ce sens et vérifié qu'on valide toujours le shema.org
tcharlss approved these changes 2 months ago
tcharlss left a comment
Owner

Super, merci

Super, merci
rastapopoulos approved these changes 2 months ago
cy.altern approved these changes 2 months ago
cerdic added 1 commit 2 months ago
cerdic force-pushed issue_6 from 9c4d749b81 to 032025da07 2 months ago
cerdic merged commit 032025da07 into master 2 months ago
cerdic deleted branch issue_6 2 months ago

Reviewers

tcharlss approved these changes 2 months ago
rastapopoulos approved these changes 2 months ago
cy.altern approved these changes 2 months ago
The pull request has been merged as 032025da07.
Sign in to join this conversation.
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.