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

Comparaison entre working copy et version #1131

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Comparaison entre working copy et version #1131

merged 2 commits into from
Dec 6, 2024

Conversation

thom4parisot
Copy link
Member

@thom4parisot thom4parisot commented Dec 5, 2024

C'était pas évident de détricoter les différentes situations.
J'ai créé des variables intermédiaires pour mieux décrire les états des composants.

J'ai aussi opté pour une approche plus accessible, où les composants ne sont pas supprimés/recréés à chaque fois mais leur état reflété par des attributs HTML (en l'occurence, hidden).

Ce qui change aussi, c'est que la working copy apparait tout le temps dans la liste des versions :

image

Peut-être que c'est un tremplin pour mieux comprendre le lien entre ce qu'on voit par défaut, et la transformation de cette working copy en version ? (à penser comme la création d'un point d'étape).

fixes #985
refs #986

Copy link

netlify bot commented Dec 5, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 4cac311
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/6752d42d8f31620008d6efa4

Copy link

netlify bot commented Dec 5, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 4cac311
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/6752d42dfb911b0008379984
😎 Deploy Preview https://deploy-preview-1131--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 Dec 5, 2024

C'est top ! Je viens de faire un petit essai et je trouve ça génial d'avoir l'état de la working copy dans la liste des versions !

@ggrossetie
Copy link
Collaborator

C'était déjà le cas mais il y a une petite zone qui n'est pas cliquable malgré le fait que le curseur indique que la zone est cliquable:

image

Au dessus et en dessous du texte comme si la marge est pas inclus dans le onClick.
Pour le reste c'est top 👍🏻

@thom4parisot
Copy link
Member Author

Ah oui, c'est le padding entre la boite et le lien :

image

@thom4parisot thom4parisot merged commit 6bd84a2 into master Dec 6, 2024
5 of 6 checks passed
@thom4parisot thom4parisot deleted the fix/985 branch December 6, 2024 10:39
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.

Comparer la working copy avec une version antérieure
3 participants