maintenant que le squelette-dist de SPIP 3.3 passe en doctype html5 et supprime les scripts de gestion d'IE < 11, cette PR propose de mettre z-core au même niveau de modernité du HTML #2

Merged
cy.altern merged 1 commits from doctype_html5 into master 1 year ago
Collaborator
There is no content yet.
cy.altern added 1 commit 2 years ago
Collaborator

+1 (a voté)

+1 (a voté)
Collaborator

+1

+1
Owner

+1 aussi (je surchargeais pour ces raisons).

+1 aussi (je surchargeais pour ces raisons).
Collaborator

+1 mais je me deande tout de même si ca vaudrait pas la peine d'avoir une version rien que pour la 3.3, pour être cohérent vec les versions de spip

+1 mais je me deande tout de même si ca vaudrait pas la peine d'avoir une version rien que pour la 3.3, pour être cohérent vec les versions de spip
cy.altern merged commit 38cb863993 into master 1 year ago
Owner

Genre SPIP 3.3 ou 4 ou je sais pas quoi est même pas releasé, mais donc y avait urgence à merger cette PR, à changer le doctype et virer les classes de support IE sans s'occuper le moins du monde de la compatibilité et en releasant ça comme une version mineure - on s'en fout bien si ça pète les sites chez les gens hein ?

Genre SPIP 3.3 ou 4 ou je sais pas quoi est même pas releasé, mais donc y avait urgence à merger cette PR, à changer le doctype et virer les classes de support IE sans s'occuper le moins du monde de la compatibilité et en releasant ça comme une version mineure - on s'en fout bien si ça pète les sites chez les gens hein ?
Poster
Collaborator

l'idée c'est que la compatibilité avec SPIP 3.2 et les doctype HTML 4 soit assurée par la branche v2.8
...après je n'avais pas la convention de nommage de branche d'où le passage par l'éphémère version spip3.2
Désolé pour le bruit : je pensais bien faire en mergeant un truc qui semblait faire l'unanimité ^^

l'idée c'est que la compatibilité avec SPIP 3.2 et les doctype HTML 4 soit assurée par la branche v2.8 ...après je n'avais pas la convention de nommage de branche d'où le passage par l'éphémère version spip3.2 Désolé pour le bruit : je pensais bien faire en mergeant un truc qui semblait faire l'unanimité ^^
Owner
  • On est bien d'accord que le commit casse les sites qui s'appuyaient sur une class iex et donc de ce fait, il ne peut pas faire l'objet d'une release mineure ?
  • Le titre de cette PR est trompeur puisque le doctype était déjà en html5 ET personalisable :
[(#VAL{_Z_DOCTYPE}|defined|?{#VAL{_Z_DOCTYPE}|constant}|sinon{<!DOCTYPE HTML>})][

maintenant c'est HTML5 ou rien, demmerdez vous. Où est le progrès ?

Donc au final, le seul intérêt de cette PR est de supprimer le vieux code de compatibilité IE11 par défaut, et du coup, je redemande :

  • est-ce que ce code est gênant par défaut ? (sachant que pour ton site, si vraiment tu n'en veux pas, tu peux avoir ta version perso du fichier structure.html)
  • est-ce qu'il y a urgence à intégrer ça ?
  • "unanimité" c'est quand même vite dit puisque en tant que mainteneur principal du plugin j'ai pas donné mon avis, et j'aimerai bien savoir si les gens ont répondu au titre de la PR ou au contenu réel ?

Sur la gestion des branches et des releases donc :

  • la convention c'est de faire des branches par version majeure, si besoin (donc v1, v2, v3), pas des branches correspondant aux versions de SPIP (pas 'spip-3.2' donc), ni des branches avec des morceaux de version (pas v2.8 donc)
  • et donc pour finir, pour moi ce commit vaut donc un passage en v3 vu qu'on supprime des compatibilités, même si c'est légitime
  • et donc on va passer le master en 3.0-dev parce qu'il y aura surement d'autres choses à mettre dans une v3, notamment si SPIP 4 intégre la balise #ICON et un jeu d'icones par défaut
  • et donc on va pas releaser de version 3 pour le moment (pas de tags ni de zip donc)
* On est bien d'accord que le commit casse les sites qui s'appuyaient sur une class `iex` et donc de ce fait, il ne peut pas faire l'objet d'une release *mineure* ? * Le titre de cette PR est trompeur puisque le doctype était *déjà* en html5 *ET* personalisable : ``` [(#VAL{_Z_DOCTYPE}|defined|?{#VAL{_Z_DOCTYPE}|constant}|sinon{<!DOCTYPE HTML>})][ ``` maintenant c'est HTML5 ou rien, demmerdez vous. Où est le progrès ? Donc au final, le seul intérêt de cette PR est de supprimer le vieux code de compatibilité IE11 par défaut, et du coup, je redemande : - est-ce que ce code est gênant par défaut ? (sachant que pour ton site, si vraiment tu n'en veux pas, tu peux avoir ta version perso du fichier `structure.html`) - est-ce qu'il y a urgence à intégrer ça ? - "unanimité" c'est quand même vite dit puisque en tant que mainteneur principal du plugin j'ai pas donné mon avis, et j'aimerai bien savoir si les gens ont répondu au titre de la PR ou au contenu réel ? Sur la gestion des branches et des releases donc : - la convention c'est de faire des branches par version majeure, si besoin (donc v1, v2, v3), pas des branches correspondant aux versions de SPIP (pas 'spip-3.2' donc), ni des branches avec des morceaux de version (pas v2.8 donc) - et donc pour finir, pour moi ce commit vaut donc un passage en v3 vu qu'on supprime des compatibilités, même si c'est légitime - et donc on va passer le master en 3.0-dev parce qu'il y aura surement d'autres choses à mettre dans une v3, notamment si SPIP 4 intégre la balise `#ICON` et un jeu d'icones par défaut - et donc on va pas releaser de version 3 pour le moment (pas de tags ni de zip donc)
The pull request has been merged as 38cb863993.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
6 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.