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

essai d'une nouvelle structure de la documentation FR #1049

Merged
merged 10 commits into from
Oct 31, 2024
Merged

Conversation

RochDLY
Copy link
Collaborator

@RochDLY RochDLY commented Oct 16, 2024

plutôt que d'avoir toute la documentation à la racine du site, la proposition suivante consiste à découper la navigation à partir de 4 sections :

  • les fonctionnalités
  • les guides d'utilisation
  • les tutoriels
  • la faq

Pour l'instant cette preuve de concept n'est que pour la partie fr de la documentation, il faudra la dupliquer sur la partie en anglais dès qu'elle sera validée

plutôt que d'avoir toute la documentation à la racine du site, la proposition suivante consiste à découper la navigation à partir de 4 sections :

- les fonctionnalités
- les guides d'utilisation
- les tutoriels
- la faq

Pour l'instant cette preuve de concept n'est que pour la partie fr de la documentation, il faudra la dupliquer sur la partie en anglais dès qu'elle sera validée
@RochDLY RochDLY self-assigned this Oct 16, 2024
Copy link

netlify bot commented Oct 16, 2024

Deploy Preview for stylo-docs ready!

Name Link
🔨 Latest commit 93e6300
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/67234f89fd90a20008f2951e
😎 Deploy Preview https://deploy-preview-1049--stylo-docs.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.

…ctions faq, tutoriels, fonctionnalites, guides_utilisation
Cette partie ne devrait pas être dans la documentation de Stylo, on peut utiliser soit le site de R30 pour ça, soit faire un blog pour Stylo, soit utiliser la fonctionnalité discussion pour épingler des nouveautés et informer les utilisateurs
@RochDLY
Copy link
Collaborator Author

RochDLY commented Oct 16, 2024

D'autres propositions :

  • on pourrait enlever la section publication et la mettre sur le site de la chaire (dans la fiche de projet Stylo par exemple). C'est du bruit en plus pour les usagers à cet endroit (ou alors il faut faire un blog de Stylo).
  • les sections contact et sources pourraient aller dans un footer
  • dans ce footer optionnel, on pourrait aussi ajouter le numéro de version de Stylo (et corréler la version de la doc à la version de Stylo)
  • la partie À propos génère du bruit dans la documentation, il vaudrait mieux mettre un colophon accessible depuis le footer et mettre à jour les contributeurs à Stylo à cet endroit

@loup-brun
Copy link
Contributor

voir #1048

@loup-brun
Copy link
Contributor

Pour remplacer les items de menu (plutôt que de combiner avec le parent), il faut voir cette partie de la documentation 11ty:
https://v2.11ty.dev/docs/data-deep-merge/

La discussion correspondante dans le développement d'11ty:
11ty/eleventy#147

… dataDeepMerge sur les données des menus de navigation
@RochDLY
Copy link
Collaborator Author

RochDLY commented Oct 18, 2024

Pour remplacer les items de menu (plutôt que de combiner avec le parent), il faut voir cette partie de la documentation 11ty: https://v2.11ty.dev/docs/data-deep-merge/

La discussion correspondante dans le développement d'11ty: 11ty/eleventy#147

Ca fonctionne bien, je remets les liens vers les versions en/fr dans chaque menu du coup

…ctionnalites, faq, guides_utilisation et tutoriels
@RochDLY RochDLY linked an issue Oct 18, 2024 that may be closed by this pull request
…dataDeepMerge(true) + reordonner le menu à gauche + certaines pages sont maintenant dans le footer
Copy link
Contributor

@loup-brun loup-brun left a comment

Choose a reason for hiding this comment

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

J'ai laissé quelques remarques sur cette PR (qui commence à être assez grosse: 35 fichiers modifiés, sans les documents en anglais). Je n'ai malheureusement pas le temps de contribuer sur le code de la documentation.

@@ -50,6 +50,9 @@ module.exports = function(eleventyConfig) {
],
});

// Empêche le comportement par défaut d'eleventy de faire un merge sur les données d'un tableau
Copy link
Contributor

Choose a reason for hiding this comment

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

Pas juste d'un tableau (Array) mais de n'importe quelle propriété héritée de la structure de données.

Copy link
Contributor

Choose a reason for hiding this comment

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

Un lien vers la documentation pourrait être ajouté dans le commentaire

Comment on lines 16 to 27
<div
class="document-footer__rights"
@html="$data.contact"
></div>
<div
class="document-footer__rights"
@html="$data.sources"
></div>
<div
class="document-footer__rights"
@html="$data.colophon"
></div>
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. La classe footer__rights ne devrait probablement pas être répétée: elle sert à indiquer la nature du contenu.
  2. est-ce que contact, sources, colophon doivent vraiment être ajoutés ici?
  3. il faudrait au moins documenter dans l'en-tête du composant quelles variables globales sont utilisées, si cet ajout est retenu.

@@ -1,5 +1,6 @@
---
title: Structurer la bibliographie
title: "Structurer la bibliographie"
date: Last modified
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-ce qu la propriété Date: Last modified est nécessaire ici? Est-ce que ce n'est pas déjà le cas pour toutes les pages? Si c'est le cas, mieux vaut éviter d'ajouter du code inutile.

Copy link
Contributor

Choose a reason for hiding this comment

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

Par souci de cohérence (s'il y en a une): est-ce que les mots des slugs ne devraient pas être séparés par des traits d'union - au lieu d'une barre en bas _? Dans la documentation, certains slugs sont sans espaces (ex. mesarticles) et d'autres comportent un trait d'union (ex. syntaxe-bibtex).

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oui je suis pour modifier cette partie. C'est dans la todo pour quand on s'occupera des contenus, ils vont bientôt être modifié du fait de la restructuration et on en profitera pour tout renommer proprement selon une seule syntaxe. Pour l'instant les contenus ont juste été ajoutés au fil des ans et sans réellement de protocole, créant ainsi une incohérence dans le nommage des fichiers

@RochDLY RochDLY changed the title WIP - essai d'une nouvelle structure de la documentation WIP - essai d'une nouvelle structure de la documentation FR Oct 21, 2024
@cgermain97
Copy link
Collaborator

Que pensez-vous de l'ajout d'un bouton (plus facilement identifiable que "Stylo" en haut à gauche) pour revenir à la page d'accueil de la documentation?

@RochDLY
Copy link
Collaborator Author

RochDLY commented Oct 22, 2024

Pour répondre à tous les commentaires, je vois qu'on a des retours sur l'UX et sur la partie graphique.
À ce stade, je ne me suis pas préoccupé de ces parties là car la PR initialement ne relevait que de la structure des informations.

Voici une proposition pour ne pas bloquer les changements de la documentation :

  1. On valide (ou non) la structure pour la partie FR de la documentation (avec les éléments du footer, est-ce qu'on les veut à cet endroit ?)
  2. Ensuite, j'efface les ajouts dans le footer pour clore la PR ou je change rapidement/temporairement le CSS pour ne pas créer de conflit sur les appellations.
  3. On merge cette PR
  4. On crée un nouveau ticket pour la structure en anglais et on réplique exactement ce qui a été fait sur la structure FR
  5. Chaque point soulevé ici fera ensuite l'objet d'un ticket et d'une PR

Est-ce que ça semble vous semble correct ? @loup-brun @cgermain97

Personnellement :

  1. La nouvelle structure tente de suivre les recommandations que l'on peut trouver sur la rédaction de documentation (cf. https://docs.divio.com/documentation-system/), elle mérite au moins d'être testée
  2. Il y a des contenus qui génèrent du bruit pour une documentation (publications, stylosophie, entre autres) et qui n'ont leur place ni dans le menu ni dans le footer.
  3. la page contact est actuellement en doublon, à la fois dans la section FAQ et dans le footer. Si on doit la retirer d'un des deux endroits, je dirais qu'on la retire de la FAQ et qu'on la garde dans le footer (ça semble nécessaire de pouvoir trouver qui contacter depuis n'importe où dans la documentation)
  4. idem pour les sources, je ne vois pas l'intérêt de dédier une page entière pour juste mettre un hyperlien "source". L'information est aussi présente dans l'index à la racine de la documentation. Peut-être qu'on peut la laisser juste à cet endroit.
  5. Plutôt qu'une page à propos dans le menu, on peut faire un colophon réduit accessible depuis le footer avec les éléments utilisés pour construire le site de la documentation et mettre à jour la liste des contributeurs de Stylo,.

@cgermain97 cgermain97 closed this Oct 23, 2024
@cgermain97 cgermain97 reopened this Oct 23, 2024
@cgermain97
Copy link
Collaborator

Oups désolée pour l'accrochage de boutons...

Je suis d'accord avec ce que tu dis Roch. Je retirerais le contact de la FAQ et au pire on peut en faire une question posée. Sinon le reste me semble beau, comme on en a discuté hier.

…changement du comportement des hyperliens et changement des noms des classes pour correspondre aux contenus. Ajout des variables globales utilisées dans le ocmposant page-footer.webc. Correction des métadonnées dans bibliographie.md. Corrections des informations dans le colophon.md. Déplacement de la page contact.md à la racine (et modifications des hyperliens)
@RochDLY RochDLY changed the title WIP - essai d'une nouvelle structure de la documentation FR essai d'une nouvelle structure de la documentation FR Oct 28, 2024
Copy link

netlify bot commented Oct 28, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 93e6300
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/67234f89fb723400080b3cc7
😎 Deploy Preview https://deploy-preview-1049--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 RochDLY merged commit e142b51 into master Oct 31, 2024
4 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.

3 participants