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

[BO - stats et dashboard] uniformisation couleur fond #3070

Conversation

hmeneuvrier
Copy link
Collaborator

Ticket

#3011

Description

Pour le dashboard,

  • il faut s'assurer qu'il y a bien une bordure et une ombre sur les widgets (voir capture ci-après)
    
  • RT et SA : passer les tableaux signalements par territoire / affectations des partenaires / ESABORA sur le design du DSFR (comme ceux sur la liste des partenaires et des étiquettes)
    
  • SA : passer le filtre territoire avec un fond gris comme ceux sur la liste des signalements
    

Pour la page stats

  • Passer les filtres avec un fond gris comme ceux sur la liste des signalements
    
  • Passer le tableau répartition par partenaire sur le design du DSFR
    
  • Pour les cartes avec les chiffres fixes : ajouter une bordure comme les widgets sur le dashboard (mais sans ombre) couleur #dddddd
    
  • Pour les graphs : ajouter une bordure couleur blue france (# 000091) comme ci-après
    

Changements apportés

  • Mise à jour de quelques styles css dans les app vue de stats et dashboard

Pré-requis

npm run build

Tests

  • Vérifier l'aspect du dashboard et des stats
  • Vérifier l'aspect des autres app vue pour éviter une régression

@@ -247,7 +247,7 @@ export default defineComponent({

<style>
#app-dashboard {
background-color: #F6F6F6;
background-color: #fff;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pas utile, tu peux supprimer

@@ -194,7 +194,7 @@ export default defineComponent({

<style>
.histo-app-stats {
background-color: var(--background-alt-grey);
background-color: #FFF;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pas utile

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tu peux aussi supprimer la règle du lien juste en dessous si tu mets une classe fr-link sur les liens (uniquement Tout réinitialiser sur cette page)

Copy link

sonarcloud bot commented Sep 20, 2024

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mes tests sont ok.
J'ai pushé des suppressions de styles inutiles. Je te laisse valider, et je mergerai ? :)

@emilschn emilschn merged commit b183154 into develop Sep 20, 2024
3 checks passed
@sfinx13 sfinx13 deleted the bugfix/3011-bo---stats-et-dashboard-uniformisation-couleur-fond branch September 24, 2024 07:15
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.

2 participants