Corrige 4811 #194

Open
RealET wants to merge 1 commits from RealET/spip:issue_4811 into master
RealET commented 3 weeks ago
cf https://core.spip.net/issues/4811
RealET added 1 commit 3 weeks ago
RealET changed title from Corrige #4811 to Corrige 4811 3 weeks ago
Owner

On peut peut être trouver une solution moins alambiquée ?

J'aurais tendance à écrire tel que : .msg-alert.type?.simple?
Ça serait clair ?

On peut peut être trouver une solution moins alambiquée ? J'aurais tendance à écrire tel que : `.msg-alert.type?.simple?` Ça serait clair ?
Owner

Ah oui c'est une indication destinée aux gens qui lisent le code, il faut que ça reste lisible. Il faudrait quelque chose de simple pour indiquer que ce sont des classes optionnelles, sans aller jusqu'à échapper donc.

La proposition de marcimat est pas mal.

Ah oui c'est une indication destinée aux gens qui lisent le code, il faut que ça reste lisible. Il faudrait quelque chose de simple pour indiquer que ce sont des classes optionnelles, sans aller jusqu'à échapper donc. La proposition de marcimat est pas mal.
Owner

Ps. : possible que j'ai fait pareil ailleurs (avec crochets), à vérifier.

Ps. : possible que j'ai fait pareil ailleurs (avec crochets), à vérifier.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
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.