-
Notifications
You must be signed in to change notification settings - Fork 13
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
Conversation
…ens absolus + pointer sur les bonnes images) + modification des nouvelles urls des pages dans les hyperliens
…json), ajout des documents pour les nouvelles fonctionnalités, début de l'écriture de la fonction corpus
… metadata, premierspas, versionner
…e fichier fr.json
…s espaces de travail + ajout des dernières publications en lien avec stylo
…, corpus, espaces de travail
✅ Deploy Preview for stylo-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for stylo-dev ready!
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
@RochDLY pour une section «nouvelles/changelog/...», il serait possible de créer une nouvelle section dans la documentation (un dossier dans Je mentionne ça juste parce que je vois qu'il y a des premières tentatives d'«alertes» sur la page d'accueil! 😉 |
…ection actualités, problème d'affichage à résoudre
…ncres sur les titres + reparation d'un lien mort dans les premierspas.md
…ontacts / premierspas / publications et sources.
…hortcode pour les images dans eleventy.js
There was a problem hiding this 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.
|
||
/* | ||
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); | ||
} |
There was a problem hiding this comment.
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
).
docs/src/en/about.md
Outdated
@@ -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]() |
There was a problem hiding this comment.
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 :
"sectionTitle": "Actualités", | ||
"sectionUrl": "/fr/actualites", | ||
"disableSearch": false, | ||
"searchIndexUrl": "/fr/search-index.json" |
There was a problem hiding this comment.
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)
"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", |
There was a problem hiding this comment.
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.
docs/src/fr/syntaxe-yaml.md
Outdated
|
||
```yaml | ||
auteur: "Bruce Wayne" | ||
|
There was a problem hiding this comment.
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é.
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) | ||
) |
There was a problem hiding this comment.
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.
…ui ne sont plus utilisées, il en reste encore ..., ajout d'un nouveau gif
…nséquences dans les documents
…e image du gestionnaire d'articles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:(
|
||
.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); | ||
} |
There was a problem hiding this comment.
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"> |
There was a problem hiding this comment.
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"
)
No description provided.