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

feat(StatementsList): add resume autoscroll button #822

Open
wants to merge 12 commits into
base: staging
Choose a base branch
from

Conversation

Akh0
Copy link
Contributor

@Akh0 Akh0 commented Apr 19, 2021

@Miragide Miragide requested a review from Betree April 21, 2021 20:55
@Miragide
Copy link
Collaborator

Pratique ! Merci @Akh0
En attente de review par @Betree

@Akh0
Copy link
Contributor Author

Akh0 commented Apr 22, 2021

@Miragide C'est pas tout à fait prêt, j'ai fait ça pour tester et voir si ça convenait à tout le monde. Si c'est ok il me reste quelques correctifs CSS, améliorer les perfs (actuellement ça rame pas mal sur les pages avec beaucoup de citations) et vérifier la compatibilité sur les différents navigateurs.

@Akh0 Akh0 marked this pull request as ready for review April 27, 2021 17:50
@Akh0 Akh0 changed the title POC: feat(StatementsList): add resume autoscroll button feat(StatementsList): add resume autoscroll button Apr 27, 2021
@Akh0
Copy link
Contributor Author

Akh0 commented Apr 27, 2021

Sur mobile le bouton "Reprendre le défilement automatique" chevauche le bouton pour ajouter une citation, c'est pas ouf.
Je suis tout ouïe si vous avez de meilleures idées sur le positionnement ou le design du bouton.

Capture d'écran Deepin_zone de sélection _20210427203049

@wdestin
Copy link
Contributor

wdestin commented Apr 28, 2021

I suggest moving the add statement button to the actions buttons under the fact-checking tab and fix the actions buttons tab with the css position property.

Example with Kickstarter :

Apr-28-2021.19-34-46.mp4

Great job btw. 👍

Edit :

This is probably the next step according to @adamferal 's suggestions about improving mobile experience :
https://www.notion.so/Project-1-drive-retention-69f7a9ea6a854b4dba6638fd5392cd41

@Miragide
Copy link
Collaborator

Bonsoir @Akh0 - peut être que ce bouton « reprendre le défilement... » n'est pas nécessaire sur mobile, mais seulement sur grand écran, quand nous sommes en deux colonnes ? Et donc que nous pouvons le désactiver sur mobile ?

Copy link
Member

@Betree Betree left a comment

Choose a reason for hiding this comment

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

This looks great! I agree that we could hide the button for mobile. But this PR is already a really nice improvement, so let me know if you don't have the time to address it now - I can merge as is and we'll follow up later.

@Akh0
Copy link
Contributor Author

Akh0 commented Jun 15, 2021

Désolé de ne pas être revenu vers vous plus tôt.
J'ai caché le bouton en CSS pour les tablettes et mobiles quand on a la vue sur 1 colonne.

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.

4 participants