Ajout d'un conseil dans la bibliothèque de médias #3252
Conversation
arnaudlevy
left a comment
arnaudlevy
left a comment
There was a problem hiding this comment.
Le texte est chouette mais il n'est pas aux clés conventionnelles
|
Suite aux retours J'ai mis le hint dans Pour la phrase, j'ai changé les textes en espérant que ca corresponde + aux "clés conventionnelles" FR : Description complémentaire de l'image. Vous pouvez donner des informations complémentaires sur l'image, utiles pour la recherche et indiquer des instructions spécifiques d'usage pour les utilisateur·rices de la médiathèque. EN : Complementary description of the image for the media library users. You can write this text to help the media library users to find an image, have complementary informations about it or have special instructions regarding the use of the image. |
|
Ajout d'un nouveau commit pour résoudre l'issue #3262 Changement de la vidéo en le contenu intégré |
config/locales/en.yml
Outdated
| embed: | ||
| title_missing: | ||
| text_html: The HTML embedding has not title. This will prevent blind people from knowing what the video is about, as mentioned in <a href="https://www.numerique.gouv.fr/publications/rgaa-accessibilite/methode-rgaa/criteres/#topic4" target="_blank">criterion 4.7 of RGAA</a>. Please add a title to the iframe. | ||
| text_html: The HTML embedding has not title. This will prevent blind people from knowing what the integrated cotnent is about, as mentioned in <a href="https://www.numerique.gouv.fr/publications/rgaa-accessibilite/methode-rgaa/criteres/#topic4" target="_blank">criterion 4.7 of RGAA</a>. Please add a title to the iframe. |
| name: Name | ||
| communication/media/localization: | ||
| internal_description: Internal description | ||
| internal_description: Internal description |
There was a problem hiding this comment.
cet espace ajouté ne sert à rien
| } %> | ||
| <%= lf.input :internal_description %> | ||
| <%= lf.input :internal_description, | ||
| hint: t('internal_description_hint') %> |
There was a problem hiding this comment.
il est inutile de forcer l'affichage d'un hint comme ça.
Si tu veux qu'un hint soit affiché il y a déjà une mécanique automatique qui le fait, à condition que tu mettes ta locale au bon endroit.
Il faut bouger ta clé pour qu'elle soit dans simple_form/hints/communication_media_localization/internal_description et ca l'affiche tout seul
| status: "Status:" | ||
| still_pending: "Your import has not been processed yet. You will receive an email as soon as it's ready." | ||
| inactivity_alert: "It seems you have been away a little bit too long. Could you please retry?" | ||
| internal_description_hint: Complementary description of the image for the media library users. You can write this text to help the media library users to find an image, have complementary informations about it or have special instructions regarding the use of the image. |
There was a problem hiding this comment.
ce n'est pas le bon endroit pour cette clé
There was a problem hiding this comment.
@sachaandre ce que veut dire @pabois c'est qu'il y a des clés conventionnelles pour les hints, de la forme suivante
simple_form:
communication_extranet:
favicon: 'Fichier .png uniquement'There was a problem hiding this comment.
(remarque bien que c'est ce que j'avais expliqué au commentaire d'avant :p )
| status: "Statut :" | ||
| still_pending: "Votre import est toujours en cours de traitement. Vous recevrez un email dès qu'il est prêt." | ||
| inactivity_alert: "Il semble que vous soyez resté sur la page un peu trop longtemps. Pouvez-vous ré-essayer ?" | ||
| internal_description_hint: Description complémentaire de l'image. Vous pouvez donner des informations complémentaires sur l'image, utiles pour la recherche et indiquer des instructions spécifiques d'usage pour les utilisateur·rices de la médiathèque. |
arnaudlevy
changed the title
|
@sachaandre tu crois que tu peux finir cette PR stp ? |
|
@sachaandre si tu préfères je la termine :) |
Pardon oui, désolé je n'ai pas ouvert github depuis un petit bout de temps |
3 participants
Vient apporter une solution pour l'issue #3242
Fix #3242
Proposition de hint
Modifications réalisées sans pouvoir le tester (je n'arrive pas à installer Osuny admin sur ma machine)