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

Permet d'utiliser Monaco à la place de CodeMirror #607

Conversation

ggrossetie
Copy link
Collaborator

editor

@ggrossetie
Copy link
Collaborator Author

ggrossetie commented Apr 7, 2022

@thom4parisot je ne sais pas trop pourquoi mais le déploiement Netlify dysfonctionne. Je pensais que c'était lié aux workspaces mais je n'en suis plus très sûr... Là, la dépendance n'a pas été installé car je crois que Netlify a réutilisé un cache (qui n'est plus à jour).

Le cache doit surement s'appliquer sur le package.json à la racine et non sur le package.json du dossier front.

Est-ce que tu vois une solution ?

(Pour débloquer, on peut purger le cache mais je n'ai pas le droit de faire cette opération)

@thom4parisot
Copy link
Member

Oui c'est relou, j'ai eu du mal avec les images Docker (l'install par workspace ne valide pas les contraintes de dépendance de la même manière qu'une install par prefix).

J'ai relancé en purgeant le cache.

@netlify
Copy link

netlify bot commented Apr 7, 2022

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 540ceb0
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/625455f8ddbe5d0009f036c5
😎 Deploy Preview https://deploy-preview-607--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 settings.

@thom4parisot
Copy link
Member

Je crois que le seul truc étrange c'est le diff : j'aurais mis la working copy à droite, pour que la couleur verte signifie ajout (de gauche à droite) et le rouge une suppression (de gauche à droite).

@ggrossetie
Copy link
Collaborator Author

Je crois que le seul truc étrange c'est le diff : j'aurais mis la working copy à droite, pour que la couleur verte signifie ajout (de gauche à droite) et le rouge une suppression (de gauche à droite).

J'ai inversé pour que ça soit comme actuellement mais je suis d'accord c'est contre-intuitif... l'affichage par défaut c'est : https://microsoft.github.io/monaco-editor/playground.html#creating-the-diffeditor-hello-diff-world

@thom4parisot
Copy link
Member

thom4parisot commented Apr 7, 2022

En discutant avec @maiwann hier, j'en avais compris que cette fonctionnalité n'est pas pas/peu utilisée car le diff est impossible à lire à cause de décalage des lignes.

Du coup je ne sais pas si le "que ça soit comme actuellement" a une raison d'être. À moins qu'il y aie une raison pour que le diff soit "inversé" par rapport à tous les outils de diff existants ?

@marviro @antoinentl @Mmellet, ça vous parle ?

@ggrossetie
Copy link
Collaborator Author

Je me rends compte qu'actuellement on affiche le texte de la copie de travail (ou de la version sélectionnée) à gauche et sur la droite on affiche un diff "inline".

Description Image
Comparaison entre versions de l'historique image
Comparaison entre la copie de travail et une version de l'historique image

Monaco permet aussi de faire un diff inline, ou un diff split avec à gauche la version originale et à droite la version modifiée. La question est de savoir ce qu'on considère comme la version originale et ce qu'on considère comme la version modifiée.
Par exemple, quand on compare une version de l'historique à la copie de travail, est-ce que la version originale est la copie de travail ? ou est-ce que c'est la version modifiée ?

On peut aussi choisir de faire du diff inline mais je pense que ça sera moins facile à lire.

@thom4parisot
Copy link
Member

Je me dis qu'on peut aussi laisser tomber la notion de workspace car on en touche ses limites.
Faudrait que je refasse une passe sur ce que ça veut dire de lancer Stylo en une seule commande, pour voir ce que ça change.

Co-authored-by: Thomas Parisot <138627+thom4parisot@users.noreply.github.com>
ggrossetie and others added 2 commits April 11, 2022 18:23
Co-authored-by: Thomas Parisot <138627+thom4parisot@users.noreply.github.com>
Co-authored-by: Thomas Parisot <138627+thom4parisot@users.noreply.github.com>
@thom4parisot thom4parisot merged commit f9cd8aa into EcrituresNumeriques:master Apr 11, 2022
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.

None yet

2 participants