feat: Permettre d’éditer un logo #4911

Merged
marcimat merged 3 commits from dev/editer_logos into master 1 month ago
Owner

Code issu du plugin Logos Plus de @RealET

Voir aussi spip/spip#3719

Une autre PR arrive dans SPIP aussi.

Code issu du plugin [Logos Plus](https://git.spip.net/spip-contrib-extensions/logosplus) de @RealET Voir aussi spip/spip#3719 Une autre PR arrive dans SPIP aussi.
marcimat added 1 commit 3 months ago
a0a4c96436 feat: Permettre d’éditer un logo
Poster
Owner

Voir aussi spip/spip#5376

Voir aussi https://git.spip.net/spip/spip/pulls/5376
b_b approved these changes 3 months ago
tcharlss reviewed 3 months ago
alt=""[
data-src-hover="(#ENV{logo_off})"]/>[(#ENV{lien}|?{</a>})]
alt="[(#ENV{alt}|sinon{#ENV{titre}}|supprimer_tags|attribut_html)]"[
Owner

Si je me souviens bien des dernières discussions sur le sujet, il était déconseillé d'utiliser le titre dans l'attribut alt.

Si je me souviens bien des dernières discussions sur le sujet, il était déconseillé d'utiliser le titre dans l'attribut alt.
b_b commented 3 months ago
Owner

Bien vu :)

Bien vu :)
tcharlss marked this conversation as resolved
Poster
Owner

Ah ? Donc j’enlève le |sinon ?

Ah ? Donc j’enlève le `|sinon` ?
Owner

sur les logos ça me choque pas qu'on mette que le alt sans fallback vers le titre, surtout qu'en général un logo est utilisé de manière illustrative et n'a pas de valeur informative, donc un alt="" par défaut me semble très bien.

(sur les documents c'est un peu plus compliqué car il y a un historique à gérer)

sur les logos ça me choque pas qu'on mette que le alt sans fallback vers le titre, surtout qu'en général un logo est utilisé de manière illustrative et n'a pas de valeur informative, donc un alt="" par défaut me semble très bien. (sur les documents c'est un peu plus compliqué car il y a un historique à gérer)
Owner

pour mémoire le ticket sur l'histoire de "alt" différent de "title" : #4615
...et effectivement ça ne me choquerait pas que pour les logos le alt puisse être vide

pour mémoire le ticket sur l'histoire de "alt" différent de "title" : https://git.spip.net/spip/medias/issues/4615 ...et effectivement ça ne me choquerait pas que pour les logos le alt puisse être vide
Poster
Owner

Puisse être vide si l’entrée "alt" n’est pas renseignée sur le document, on est d’accord ?

Puisse être vide si l’entrée "alt" n’est pas renseignée sur le document, on est d’accord ?
marcimat force-pushed dev/editer_logos from a0a4c96436 to 4be880b370 3 months ago
Poster
Owner

Je viens d’enlever le sinon{titre} déjà.

Je viens d’enlever le `sinon{titre}` déjà.
Owner

Ça me semble bon là du coup @marcimat

Ça me semble bon là du coup @marcimat
tcharlss approved these changes 3 months ago
tcharlss left a comment
Owner

Ok pour medias.

Par contre pour la PR du core j'ai toujours des questions/remarques.

Ok pour medias. Par contre pour la PR du core j'ai toujours des questions/remarques.
cy.altern approved these changes 3 months ago
Poster
Owner

Alors du coup, si le 'titre' n’est pas mis dans le 'alt', mais qu’on affiche le titre sous le logo en colonne, est-ce qu’il faut / faudrait aussi montrer un alt éventuel s’il est renseigné ?

Le titre est visible

Alors du coup, si le 'titre' n’est pas mis dans le 'alt', mais qu’on affiche le titre sous le logo en colonne, est-ce qu’il faut / faudrait aussi montrer un alt éventuel s’il est renseigné ? ![Le titre est visible](https://git.spip.net/attachments/ebbfa3e3-3757-49c2-ac9f-805a0319fc8c)
Owner

Oui ça semble pas mal, ça permettrait de vérifier en un coup d'oeil la présence ou non du alt sans avoir à cliquer sur modifier.

Oui ça semble pas mal, ça permettrait de vérifier en un coup d'oeil la présence ou non du alt sans avoir à cliquer sur modifier.
marcimat added 2 commits 1 month ago
marcimat added 2 commits 1 month ago
Poster
Owner

Avec 8fceae0830 j’ajoute un test assez basique pour ne pas permettre d’illustrer un logo.

Faudrait il passer par une autorisation ? On on verra plus tard si le besoin d’en fait sentir ?

Avec 8fceae0830d j’ajoute un test assez basique pour ne pas permettre d’illustrer un logo. Faudrait il passer par une autorisation ? On on verra plus tard si le besoin d’en fait sentir ?
b_b commented 1 month ago
Owner

On on verra plus tard si le besoin d’en fait sentir ?

+1, faisons simple dans un premier temps :)

> On on verra plus tard si le besoin d’en fait sentir ? +1, faisons simple dans un premier temps :)
marcimat merged commit 6bd79d8933 into master 1 month ago
marcimat deleted branch dev/editer_logos 1 month ago

Reviewers

b_b approved these changes 3 months ago
tcharlss approved these changes 3 months ago
cy.altern approved these changes 3 months ago
The pull request has been merged as 6bd79d8933.
Sign in to join this conversation.
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.