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

(PC-31593)[PRO] refactor: Remove titles component and replace occurre… #13921

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

GuillaumeMgz
Copy link
Contributor

@GuillaumeMgz GuillaumeMgz commented Sep 2, 2024

…nces with h1.

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-31593

Objectif
Suppression du composant Titles qui n'a pas d'autre utilité que d'afficher un h1. Ce ui évite d'avoir à écrire la doc accessibilité pour ce composant.
Il y a une utilisation de la prop action que j'ai remplacé explicitement.

Au passage dans le 2e commit, nettoyage du style global section.

Vérifications

  • J'ai écrit les tests nécessaires
  • J'ai mis à jour le fichier des plans de tests du portail pro si nécessaire
  • J'ai mis à jour la liste des routes et des titres de pages du portail pro si j'en ai rajouté/modifié ou supprimé une.
  • J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
  • J'ai ajouté des screenshots pour d'éventuels changements graphiques

Copy link
Contributor

github-actions bot commented Sep 2, 2024

Visit the preview URL for this PR (updated for commit 22f7f38):

https://pc-pro-testing--pr13921-pc-31593-remove-titl-s550ilpq.web.app

(expires Wed, 04 Sep 2024 14:12:53 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1

@GuillaumeMgz GuillaumeMgz force-pushed the pc-31593-remove-titles-component branch from aa336ae to 22f7f38 Compare September 2, 2024 14:02
@GuillaumeMgz GuillaumeMgz merged commit c1507b7 into master Sep 3, 2024
18 checks passed
@GuillaumeMgz GuillaumeMgz deleted the pc-31593-remove-titles-component branch September 3, 2024 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants