depuis 1a903d57b6, la class css-loading est sur le conteneur. Donc en cas d'erreur il faut l'enelever du conteneur, pas du form #2

Closed
maieul wants to merge 2 commits from css_erreurs into master
maieul commented 2 years ago
Collaborator

hop @cerdic. Tout est dans le message de commit.
Pour voir le problème : crayonner en generant une erreur.

Avant ce commit, on ne pouvait pas voir le message d'erreur.

hop @cerdic. Tout est dans le message de commit. Pour voir le problème : crayonner en generant une erreur. Avant ce commit, on ne pouvait pas voir le message d'erreur.
Owner

Plutôt alors un

crayon
  .removeClass('crayon-loading')
  ...

non ?

Parce que le principe de 1a903d57b6 c'était justement de plus manipuler en dur les styles mais uniquement la classe, ce qui permet plus facilement de personaliser le rendu des crayons
(et en l'occurence moi je mets aussi du blur sur le loading, pas que de l'opacity)

Plutôt alors un ``` crayon .removeClass('crayon-loading') ... ``` non ? Parce que le principe de https://git.spip.net/spip-contrib-extensions/crayons/commit/1a903d57b6395d6b9803835485e022cec387d84b c'était justement de plus manipuler en dur les styles mais uniquement la classe, ce qui permet plus facilement de personaliser le rendu des crayons (et en l'occurence moi je mets aussi du blur sur le loading, pas que de l'opacity)
Poster
Collaborator

a oui d'accord. Il restait encore du coup un peu de css en dur pour le erreurs, mais il faudrait précisement le supprimer. Je te refais un PR.

a oui d'accord. Il restait encore du coup un peu de css en dur pour le erreurs, mais il faudrait précisement le supprimer. Je te refais un PR.
Poster
Collaborator

Hum, oui oki.

Bon ca veut dire que je dois sérieusement me mettre au travail autour du versionnement des réponses formidable, sachant que c'est un objet bancale en terme de structure.

Hum, oui oki. Bon ca veut dire que je dois sérieusement me mettre au travail autour du versionnement des réponses formidable, sachant que c'est un objet bancale en terme de structure.
maieul closed this pull request 2 years ago
Owner

non non la fonction xxx_revision n'est pas lié au versionnement mais à la mise à jour. C'est l'ancien nom conventionnel dans spip avant le modifier

non non la fonction `xxx_revision` n'est pas lié au versionnement mais à la mise à jour. C'est l'ancien nom conventionnel dans spip avant le `modifier`
maieul reopened this pull request 2 years ago
Poster
Collaborator

Oups, j'avais fermé ceci mais c'était une erreur. Cf. remarque en #8. Cela étant les remarques de @cerdic restent valable, faut nettoyer.

Oups, j'avais fermé ceci mais c'était une erreur. Cf. remarque en #8. Cela étant les remarques de @cerdic restent valable, faut nettoyer.
alexis.pellicier added 1 commit 2 years ago
Owner

bon je sais pas trop le mic mac avec cette branche et le merge d'alexis, mais j'ai enfin pris le temps de regarder.
C'est fixé avec 72d12ac sachant qu'on ne passait jamais dans ce bout de code a cause d'une typo sur la variable, ce qui est aussi corrigé.

J'en ai profité pour tester/corriger la gestion d'erreur cf caf91eaba6 et 9955b95851

bon je sais pas trop le mic mac avec cette branche et le merge d'alexis, mais j'ai enfin pris le temps de regarder. C'est fixé avec 72d12ac sachant qu'on ne passait jamais dans ce bout de code a cause d'une typo sur la variable, ce qui est aussi corrigé. J'en ai profité pour tester/corriger la gestion d'erreur cf caf91eaba6f36689cd45fd967340242d6ff0070c et 9955b95851b65464b628a0ded694574a68c02d55
cerdic closed this pull request 2 years ago
cerdic deleted branch css_erreurs 2 years ago
Poster
Collaborator

merci :)

merci :)
Collaborator

Le merge ce n'est pas ce que je voulais faire. Désolé je patauge :(

Merci en tout cas!

Le merge ce n'est pas ce que je voulais faire. Désolé je patauge :( Merci en tout cas!
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.