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

903 mise a jour documentation #939

Merged
merged 40 commits into from
Oct 5, 2023
Merged

Conversation

RochDLY
Copy link
Collaborator

@RochDLY RochDLY commented Sep 25, 2023

No description provided.

@netlify
Copy link

netlify bot commented Sep 25, 2023

Deploy Preview for stylo-docs ready!

Name Link
🔨 Latest commit f35a025
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/651ee34b1736ad0008924d2d
😎 Deploy Preview https://deploy-preview-939--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.

@netlify
Copy link

netlify bot commented Sep 25, 2023

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 881f88f
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/65118a8e3093340008db0bb6
😎 Deploy Preview https://deploy-preview-939--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.

… monaco, premiers pas, syntaxe bibtex, syntaxe yaml)
… et modification des alert-block dans l'index
…commencer avec, les contacts pour signaler les bugs passent de sources à contact.md, ajout d'un article dans les publications, aerer la page versionner.md
@loup-brun
Copy link
Contributor

loup-brun commented Sep 29, 2023

@RochDLY pour une section «nouvelles/changelog/...», il serait possible de créer une nouvelle section dans la documentation (un dossier dans docs/src/<section>), avec son fichier de métadonnées éponyme (docs/src/<section>/<section>.json). Il serait ainsi possible d'avoir une section avec une navigation cliquable (comme c'est actuellement le cas dans la barre supérieure: Stylo > Documentation > Titre de la page).

Je mentionne ça juste parce que je vois qu'il y a des premières tentatives d'«alertes» sur la page d'accueil! 😉

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.

Voir les commentaires ciblés. Je recommanderais de rapidement segmenter la PR et de faire une liste dans l'issue correspondante.

Comment on lines 275 to 316

/*
Overlay images
*/

.overlay {
position: fixed;
z-index: 99;
top: 0;
left: 0;
width: 100%;
height: 100%;
background: rgba(0,0,0,0.9);
display: flex;
align-items: center;
text-align: center;
visibility: hidden; /* reset on :target */
opacity: 0; /* reset on :target */
transition: opacity .3s;
}

.overlay img{
width: auto;
height: auto;
transform: scale(0.95);
transition: transform .3s;
max-height: 40vh;
object-fit: unset;
margin-left: auto;
margin-right: auto;
}

.overlay:target {
visibility: visible;
outline: none;
cursor: default;
opacity: 1;
}

.overlay:target img {
transform: scale(1.5);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

@RochDLY ces styles ne devraient pas figurer dans modern-normalize.css. La mission d'un « normalize » est de faire un « reset » – remettre des styles généraux à une base cohérente, pas d'ajouter des nouvelles fonctionnalités.

Ces styles devraient être déplacés dans une feuille CSS séparée, ex. image-overlay.css ou à tout le moins styles/global.css.
Mieux : La plupart des styles sont encapsulés dans des composantes. Il semble que les images cliquables soient tous encapsulés dans la prose. Ma recommandation serait donc de déplacer ces styles dans le bloc <style> du composant prose-block.webc (docs/src/_includes/components/prose-block.webc).

@@ -54,8 +54,11 @@ The researchers participating in the project are as follows:
- [Servanne Monjour](https://ecrituresnumeriques.ca/fr/Equipe/Servanne-Monjour-)
- [Nicolas Sauret](https://ecrituresnumeriques.ca/fr/Equipe/Nicolas-Sauret)
- [Marcello Vitali-Rosati](https://ecrituresnumeriques.ca/fr/Equipe/Marcello-Vitali-Rosati-)
- [Roch Delannay](https://ecrituresnumeriques.ca/en/Equipe/Roch-Delannay)
- [Louis-Olivier Brassard]()
Copy link
Contributor

Choose a reason for hiding this comment

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

Mon URI personnel pourrait convenir, en attendant mon profil sur ÉcrituresNumériques.ca :

https://www.loupbrun.ca

"sectionTitle": "Actualités",
"sectionUrl": "/fr/actualites",
"disableSearch": false,
"searchIndexUrl": "/fr/search-index.json"
Copy link
Contributor

Choose a reason for hiding this comment

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

Attention : l'index de recherche est-il alimenté par des nouveaux contenus de la section actualités? (à tester)

Comment on lines 2 to 8
"layout": "docs",
"useSideMenu": true,
"siteTitle": "Stylo",
"lang": "fr",
"description": "Stylo est un éditeur de texte simplifiant la rédaction et l'édition d'articles scientifiques en Sciences Humaines et Sociales.",
"rights": "Licence <a href=\"https://creativecommons.org/licenses/by-sa/4.0/\">Creative Commons [CC BY-SA 4.0]</a> – Chaire de recherche du Canada sur les écritures numériques",
"sectionTitle": "Actualités",
Copy link
Contributor

Choose a reason for hiding this comment

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

Attention : ces paramètres ne devraient pas être répétés dans les sections enfant, à moins d'être signifiants et surtout différents.


```yaml
auteur: "Bruce Wayne"

Copy link
Contributor

Choose a reason for hiding this comment

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

Attention aux lignes vides superflues dans les blocs de texte préformaté.

Comment on lines +8 to +35
const pluginTOC = require('eleventy-plugin-toc');
const markdownIt = require('markdown-it');
const markdownItAnchor = require('markdown-it-anchor');

// des options pour le rendu avec markdown-it
const mdOptions = {
html: true,
breaks: false,
linkify: true,
typographer: true
}

// constante pour les ancres dans les titres
const mdAnchorOpts = {
permalink: true,
permalinkClass: 'anchor-link',
permalinkSymbol: '#',
level: [1, 2, 3, 4]
};

module.exports = function(eleventyConfig) {

//Markdown : ajout des ancres sur les titres et la classe 'anchor-link'
eleventyConfig.setLibrary(
'md',
markdownIt(mdOptions)
.use(markdownItAnchor, mdAnchorOpts)
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Petite remarque sur l’étendue (scope-creep) de cette PR : cette mise à jour de la documentation ratisse très large au point qu’une révision devient difficile (la révision du contenu modifié partout) et multifacette (contenu ainsi que structure de programmation).

Ici, l'ajout de paquets modifie potentiellement le rendu du contenu HTML et ajoute des dépendances supplémentaires (à considérer avec soin).

Je recommanderais de différer l'ajout d'éléments structurels de changements au contenu de la documentation (autrement dit: modifications markdown d'une part; modifications 11ty d'autre part).

L'ajout de # (ancres aux en-têtes) induit des considérations visuelles (design visuel plus chargé et à ajuster).
Cela pourrait facilement faire l'objet d'une PR séparée.

@RochDLY RochDLY merged commit 30b8583 into master Oct 5, 2023
3 checks passed
@RochDLY RochDLY deleted the 903-mise-a-jour-documentation branch October 5, 2023 16:49
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.

:(

Comment on lines +423 to +439

.toc {
position: fixed;
}

.toc ul {
list-style: none;
}

.toc a:link, a:visited {
text-decoration: none;
color: var(--couleur-police);
}

.toc ul > li > a:hover, a:active {
color: var(--couleur-lien, inherit);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Ces styles devraient figurer dans le composant toc (lequel devrait d'ailleurs, d'après la convention des web components, comporter un trait d'union pour éviter les conflits futurs avec des éléments HTML).

@@ -171,10 +171,13 @@
</div>

<div class="docs-layout__center-wrapper | grid-area--page-sidebar">
<nav class="docs-layout__center-sidebar">
<div class="docs-layout__center-sidebar">
Copy link
Contributor

Choose a reason for hiding this comment

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

Cela peut rester un nav sémantique et être assigné à un composant avec les bons attributs (webc:is="nom-du-composant")

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