La fonction montant_formater introduit plein de retour ligne #11

Closed
opened 5 months ago by cerdic · 2 comments
cerdic commented 5 months ago
Owner

Ici donc
https://git.spip.net/spip-contrib-extensions/intl/src/branch/master/intl_fonctions.php#L174

et comme dans le plugin commandes on a la bonne idée d'avoir une règle CSS qui dit

table.bordereau td.prix {
  white-space: pre;
}

autant dire que c'est la fête au village : tous les prix s'affichent sur 3 lignes, soulignés en dotted car on est dans un abbr...

Bref. Ça peut arriver n'importe où, et je pense qu'il serait prudent que cette fonction n'envoie pas d'espace surnuméraire

Tout collé, comme par exemple

	$montant_formate =
		"<span class=\"$classe\" data-montant-nombre=\"$montant\" data-montant-devise=\"$devise\">"
		. "$montant_formate"
		. "<meta itemprop=\"price\" content=\"$montant\" />"
		. "<meta itemprop=\"priceCurrency\" content=\"$devise\" />"
		. "</span>";

me paraitrait plus judicieux

Ici donc https://git.spip.net/spip-contrib-extensions/intl/src/branch/master/intl_fonctions.php#L174 et comme dans le plugin commandes on a la bonne idée d'avoir une règle CSS qui dit ``` table.bordereau td.prix { white-space: pre; } ``` autant dire que c'est la fête au village : tous les prix s'affichent sur 3 lignes, soulignés en dotted car on est dans un abbr... Bref. Ça peut arriver n'importe où, et je pense qu'il serait prudent que cette fonction n'envoie pas d'espace surnuméraire Tout collé, comme par exemple ``` $montant_formate = "<span class=\"$classe\" data-montant-nombre=\"$montant\" data-montant-devise=\"$devise\">" . "$montant_formate" . "<meta itemprop=\"price\" content=\"$montant\" />" . "<meta itemprop=\"priceCurrency\" content=\"$devise\" />" . "</span>"; ``` me paraitrait plus judicieux
Owner

Heu oui, avec le recul c'était une fausse bonne idée.
Sus aux \n et aux \t

Heu oui, avec le recul c'était une fausse bonne idée. Sus aux `\n` et aux `\t`
Poster
Owner

corrigé par 87da8c23c1 donc

corrigé par https://git.spip.net/spip-contrib-extensions/intl/commit/87da8c23c128293d7f736de394e6f1067db13138 donc
cerdic closed this issue 5 months ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.