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

feat: affiche une barre de statistique dans le pied de page #1091

Merged

Conversation

ggrossetie
Copy link
Collaborator

resolves #1087

Copy link

netlify bot commented Nov 7, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 11cd3a6
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/6736058d63729e00089271a6

Copy link

netlify bot commented Nov 7, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 11cd3a6
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/6736058df71856000892bf6e
😎 Deploy Preview https://deploy-preview-1091--stylo-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RochDLY
Copy link
Collaborator

RochDLY commented Nov 8, 2024

Je remets également pour cette PR ce dont nous avons discuté en privé : la barre des statistiques en bas est très bien, par contre nous nous sommes demandés si le scrool pour afficher le footer n'était pas dérangeant. Si oui, est-ce qu'il est possible de mettre les informations du footer ailleurs dans la page (par exemple dans le bandeau noir en en-tête, afin de supprimer ce footer ? 

@ggrossetie
Copy link
Collaborator Author

Je remets également pour cette PR ce dont nous avons discuté en privé : la barre des statistiques en bas est très bien, par contre nous nous sommes demandés si le scrool pour afficher le footer n'était pas dérangeant. Si oui, est-ce qu'il est possible de mettre les informations du footer ailleurs dans la page (par exemple dans le bandeau noir en en-tête, afin de supprimer ce footer ? 

Comme ce n'est pas directement lié, j'ai crée une discussion sur le sujet : #1095

Est-ce qu'on déploie ce changement sur stylo-dev afin de collecter des retours supplémentaires ?

@ggrossetie
Copy link
Collaborator Author

Nous avons décidé de supprimer le pied de page sur la page de l'éditeur de texte afin d'éviter d'avoir une barre de scroll sur la fenêtre.

- crée des modules CSS pour chaque composant
- supprime les styles non utilisés
- supprime la barre de scroll verticale sur l'éditeur
@ggrossetie ggrossetie merged commit 88046e0 into EcrituresNumeriques:master Nov 14, 2024
12 checks passed
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.

Déplacer les statistiques d'un article en pied de page
2 participants