Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix du problème des numéros de téléphone #650

Merged
merged 8 commits into from
Oct 22, 2024
Merged

Conversation

Olivia206
Copy link
Contributor

@Olivia206 Olivia206 commented Oct 9, 2024

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Magali a repéré un truc étrange sur Re-akt (issue #5) : le numéro de tel affiche dans son url un id étrange. Ça vient de la protection automatique de Hugo.

Il faut savoir que la donnée est comme ceci :
Capture d’écran 2024-10-09 à 18 12 54

Le "tel:" déclenche la protection d'hugo (j'ignore s'il y avait une bonne raison pour cet usage, on ne fait ça nulle part ailleurs), j'ai donc rajouté un safeURL

J'en ai profité pour ajouter des title accessibles, en ajoutant quelques traductions bienvenues et en utilisant celles configurées pour les blocs contact et les liens réseaux du footer.

En regardant l'appel du partial, j'ai remarqué qu'on avait 2 partials vraiment similaire, j'ai donc factorisé en 1 seul "commons/contact-details" → la seule différence c'est que le .phone de l'organisation n'est plus après l'addresse : est-ce que je rajoute un check qui permet de l'afficher à la suite de l'addresse ?

Je laisse en draft car il reste :

  • valider les traductions et si tout est ok ajouter celles anglaises
  • tester sur plusieurs sites pour s'assurer que tout fonctionne
  • Question : faut-il différencier dans les title alternatifs le tel du tel pro ?

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

Référence (ticket et/ou figma)

osunyorg/reakt-site#5 (comment)

URL de test sur example.osuny.org

http://localhost:1313/fr/equipe/pierre-andre-boissinot/

URL de test du site Re-akt

http://localhost:1313/equipe/nicolas-pierret/

URL de test du site IUT Bordeaux Montaigne

http://localhost:1313/notre-institut/ressources/equipe/clotilde-de-montgolfier/

Screenshots

Capture d’écran 2024-10-10 à 10 26 17 Capture d’écran 2024-10-10 à 10 26 20

@Olivia206 Olivia206 added bug Something isn't working Incidence faible 😌 Risque faible de régression sur les sites adjustment a11y labels Oct 9, 2024
@arnaudlevy
Copy link
Member

Est-ce qu'on est obligés d'être si descriptifs que "Contactez Nicolas Pierret par téléphone au ..." ?
Ne peut-on pas simplement décrire "Appeler le ..." ?

@Olivia206 Olivia206 changed the title Phone numbers safety Fix du problème des numéros de téléphone Oct 10, 2024
@Olivia206
Copy link
Contributor Author

Changement du côté des données : admin #2303

@Olivia206 Olivia206 marked this pull request as draft October 10, 2024 12:35
@Olivia206 Olivia206 marked this pull request as ready for review October 11, 2024 12:27
@alexisben alexisben merged commit 8b97200 into main Oct 22, 2024
3 checks passed
@alexisben alexisben deleted the phone-numbers-safety branch October 22, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y adjustment bug Something isn't working Incidence faible 😌 Risque faible de régression sur les sites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants