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

Ajouter des ancres permettant un retour aux sections #1801

Closed
Tracked by #1782
maximeperraultdev opened this issue Oct 28, 2024 · 3 comments · Fixed by #1865, #1913 or #1934
Closed
Tracked by #1782

Ajouter des ancres permettant un retour aux sections #1801

maximeperraultdev opened this issue Oct 28, 2024 · 3 comments · Fixed by #1865, #1913 or #1934

Comments

@maximeperraultdev
Copy link
Collaborator

No description provided.

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Nov 26, 2024

@maximeperraultdev c'est bon ça marche bien, le seul point c'est qu'elles mettent du temps à s'actualiser (parfois plusieurs secondes) : quand je clique sur une ancre, elle ne disparaît pas tout de suite, et les ancres "opposées" mettent également un petit moment à apparaître. Mais je ne sais pas si c'est parce que mon ordi est particulièrement lent ou non, il faudrait voir côté CACEM si ça le fait aussi.
Cf. vidéo ci-dessous
https://github.com/user-attachments/assets/51725336-718e-4df7-8c5c-31e059d4a854

Et sinon il y a juste un petit point UI : j'ai l'impression que les marges ne sont pas tout à fait bonnes, j'ai l'impression que les ancres sont un peu trop "vers l'extérieur". Cf. marges des maquettes ci-dessous :
Image

@AdelineCelier AdelineCelier reopened this Nov 26, 2024
@AdelineCelier AdelineCelier moved this from To Review in INT to Todo in Kanban MonitorEnv Nov 26, 2024
@maximeperraultdev maximeperraultdev moved this from Todo to In progress in Kanban MonitorEnv Nov 28, 2024
@claire2212 claire2212 moved this from In progress to In Code Review in Kanban MonitorEnv Dec 6, 2024
@claire2212 claire2212 moved this from In Code Review to To Review in INT in Kanban MonitorEnv Dec 6, 2024
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Dec 9, 2024

@maximeperraultdev on n'y est pas encore tout à fait :-/
J'ai fait une vidéo pour te montrer le comportement sur mon ordi :

  • quand je déplie une section qui fait passer sous la ligne de flottaison les sections en-dessous, les ancres ne s'affichent pas. C'est quand je commence à scroller dans la colonne que les ancres s'affichent. Ce n'est pas dramatique mais je trouverais plus logique qu'elles s'affichent dès la disparition des sections correspondantes. 👍🏻
  • (Side Window - Soucis de rendering monitorfish#3807 y a parfois des ancres qui restent affichées même quand on a replié les sections.
2024-12-09.16-08-28.mp4
  • j'ai l'impression que le point UI remonté ci-dessus n'est pas tout à fait réglé : les ancres débordent encore un peu sur les bandeaux du haut et du bas – c'est chiant à régler ou pas ? 👍🏻
  • il faut mettre un contour de 1 px en Gainsboro sur les ancres, pour qu'elles se détachent encore mieux du fond. 👍🏻

Image

@AdelineCelier AdelineCelier moved this from To Review in INT to Todo in Kanban MonitorEnv Dec 9, 2024
@AdelineCelier AdelineCelier reopened this Dec 9, 2024
@maximeperraultdev maximeperraultdev moved this from Todo to To be deployed in INT for review in Kanban MonitorEnv Dec 11, 2024
@claire2212 claire2212 moved this from To be deployed in INT for review to To Review in INT in Kanban MonitorEnv Dec 19, 2024
@AdelineCelier
Copy link
Collaborator

Les points que tu as cochés sont bien réglés, et je n'ai pas revu d'ancres qui restaient affichées, donc on dit que c'est bon, je passe le ticket !

@AdelineCelier AdelineCelier moved this from To Review in INT to To deploy in PROD in Kanban MonitorEnv Dec 19, 2024
@claire2212 claire2212 moved this from To deploy in PROD to Done in Kanban MonitorEnv Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment