#99 issue_4651: suppression de l'option "Se limiter au HTML4 sur le site public"

Merged
cerdic merged 3 commits from issue_4651 into master 1 week ago
Owner

Par défaut SPIP 3.3 est en HMTL 5
Le plugin historique_spip32_html4 permet de rétablir le comportement de l'option "Se limiter au HTML4 sur le site public" via l'ajout de la constante _VERSION_HTML_4_MAX

Par défaut SPIP 3.3 est en HMTL 5 Le plugin [historique_spip32_html4](https://git.spip.net/spip-contrib-extensions/historique_spip32_documents) permet de rétablir le comportement de l'option *"Se limiter au HTML4 sur le site public"* via l'ajout de la constante _VERSION_HTML_4_MAX
cy.altern added 2 commits 3 weeks ago
cy.altern changed title from issue_4651 to issue_4651: suppression de l'option "Se limiter au HTML4 sur le site public" 3 weeks ago
nicod_ commented 3 weeks ago
Poster
Owner

Super !

« Un petit pas pour l'homme, un grand pas pour le HTML » 😄

On n'en profiterait pas pour tordre le coup une fois pour toute à ce ticket là aussi ?
https://core.spip.net/issues/2381

Super ! *« Un petit pas pour l'homme, un grand pas pour le HTML »* 😄 On n'en profiterait pas pour tordre le coup une fois pour toute à ce ticket là aussi ? https://core.spip.net/issues/2381
Poster
Owner

Ça fait longtemps que je dis que ça devrait être en constante donc +1 sur le principe, mais pas sur l'implémentation proposée là, comme dit dans le ticket : le nom de variable était générique et donc pérenne pour le futur si on doit utiliser d'autre chose (htmlX ou whatever variante) : aucune nécessité de régresser en retirant l'aspect générique où on peut mettre la valeur qu'on veut.
https://core.spip.net/issues/4651#note-3

_VERSION_HTML tout court devrait suffire même (pourquoi ce "max" même avant ?) puisqu'il n'y a qu'une seule version en doctype dans chaque page.

Donc define('_VERSION_HTML', 'html5') et ça laisse tout liberté pour le futur.

Ça fait longtemps que je dis que ça devrait être en constante donc +1 sur le principe, mais pas sur l'implémentation proposée là, comme dit dans le ticket : le nom de variable était générique et donc pérenne pour le futur si on doit utiliser d'autre chose (htmlX ou whatever variante) : aucune nécessité de régresser en retirant l'aspect générique où on peut mettre la valeur qu'on veut. https://core.spip.net/issues/4651#note-3 `_VERSION_HTML` tout court devrait suffire même (pourquoi ce "max" même avant ?) puisqu'il n'y a qu'une seule version en doctype dans chaque page. Donc `define('_VERSION_HTML', 'html5')` et ça laisse tout liberté pour le futur.
cy.altern added 1 commit 3 weeks ago
Poster
Owner

c'est réglé par f956bf7bb7

c'est réglé par https://git.spip.net/spip/spip/commit/f956bf7bb71e06c9f67197a89b1930f2bcbb85e4
Poster
Owner

On n'en profiterait pas pour tordre le coup une fois pour toute à ce ticket là aussi ?
https://core.spip.net/issues/2381

La PR spip/textwheel#3 les +1 :)

> On n'en profiterait pas pour tordre le coup une fois pour toute à ce ticket là aussi ? > https://core.spip.net/issues/2381 La PR https://git.spip.net/spip/textwheel/pulls/3 les +1 :)
b_b commented 3 weeks ago
Poster
Owner

Super, pour compléter avant merge il faut aussi penser à retirer les chaînes de langues utilisées dans le formulaire configurer_moderniseur.

Super, pour compléter avant merge il faut aussi penser à retirer les chaînes de langues utilisées dans le formulaire `configurer_moderniseur`.
cerdic merged commit 1f5d193230 into master 1 week ago
b_b commented 1 week ago
Poster
Owner

Ha ben tant pis pour commentaire, je ferai le ménage dans les items de langue moi même alors...

e4fc24fea5

~~Ha ben tant pis pour commentaire, je ferai le ménage dans les items de langue moi même alors...~~ https://git.spip.net/spip/spip/commit/e4fc24fea500da17b73169b7f3ebdf6020e7b407
cerdic commented 1 week ago
Poster
Owner

Intégré et chaines de langues supprimées par e4fc24fea5

Intégré et chaines de langues supprimées par e4fc24fea5
b_b deleted branch issue_4651 1 week ago
The pull request has been merged as 1f5d193230.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.