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

Problème d’affichage de tableaux en format mobile #4427

Closed
pierre-24 opened this issue Aug 2, 2017 · 14 comments
Closed

Problème d’affichage de tableaux en format mobile #4427

pierre-24 opened this issue Aug 2, 2017 · 14 comments
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème

Comments

@pierre-24
Copy link
Member

pierre-24 commented Aug 2, 2017

Vu sur le forum:

Quand la fenêtre est assez grande, pas de soucis, tout s'affiche bien.

Historique des éditions du message.

Mais sur une fenêtre à largeur réduite, le tableau est tronqué, et les boutons pour voir / restaurer deviennent ainsi inaccessibles.

Tableau tronqué.

Ce n'est pas de la plus haute importance.
Quoique, je ne sais pas ce que ça peut rendre sur mobile.

@pierre-24 pierre-24 added C-Front Concerne l'interface du site S-BUG Corrige un problème labels Aug 2, 2017
@pierre-24 pierre-24 added this to the v24 milestone Aug 2, 2017
@pierre-24
Copy link
Member Author

pierre-24 commented Aug 2, 2017

Et, d'après @vhf, pareil sur la gestion des alertes:

screenshot_20170802-110833

@pierre-24 pierre-24 changed the title Problème d’affichage de l’historique d’édition des messages sur le forum Problème d’affichage de tableaux en format mobile Aug 2, 2017
@motet-a
Copy link
Contributor

motet-a commented Aug 2, 2017

Ça semble voulu puisque ici et il y a un class="wide".

@pierre-24
Copy link
Member Author

C'est surtout que le dévellopeur a testé sur son pc, à mon avis ^^

@motet-a
Copy link
Contributor

motet-a commented Aug 2, 2017

@pierre-24 C’est à dire ? Le class="wide" sur la dernière colonne veut dire « n’afficher que sur écran large » :

https://github.com/zestedesavoir/zds-site/blob/dev/assets/scss/base/_helpers.scss#L72

Par « Ça semble voulu », je veut dire que ce n’est pas un bug, mais que le truc est conçu comme ça (mais je ne dis pas que c’est très bien conçu, un tableau n’est pas forcément le bon choix ici).

@pierre-24
Copy link
Member Author

En effet.

Mais je pense que les gens aimeraient pouvoir y accéder aussi en format mobile :)

@pierre-24 pierre-24 changed the title Problème d’affichage de tableaux en format mobile [beta][v24] Problème d’affichage de tableaux en format mobile Aug 2, 2017
@motet-a
Copy link
Contributor

motet-a commented Aug 2, 2017

Je pense aussi 😃

@artragis
Copy link
Member

artragis commented Aug 2, 2017

j'ai tendance à penser que ça n'est pas propre à la béta tout ça.

@vhf
Copy link
Contributor

vhf commented Aug 2, 2017

Ça ne l'est pas. Tu peux enlever la milestone v24. Comme je le disais, c'est pareil sur les alertes sur la prod.

@pierre-24 pierre-24 removed this from the v24 milestone Aug 2, 2017
@pierre-24 pierre-24 changed the title [beta][v24] Problème d’affichage de tableaux en format mobile Problème d’affichage de tableaux en format mobile Aug 2, 2017
@pierre-24
Copy link
Member Author

Voilà qui est fait.

@rezemika
Copy link
Contributor

rezemika commented Aug 3, 2017

Je m'en occupe. :-)

Quelles sont les pages affectées ? Seulement celle-ci ou toutes celles qui ont des tableaux avec des boutons ?

Edit : Ce n'est pas le même problème, mais je trouves que les boutons sont très "serrés" sur les petits écrans. Il faudrait peut-être ajouter du padding entre-eux ?

@motet-a
Copy link
Contributor

motet-a commented Aug 3, 2017

@rezemika Ici et il y a un class="wide".

Mais pour corriger le problème correctement, je pense qu’il faut mieux remplacer tout le tableau par quelque chose de responsive.

Concernant les boutons, je pense que tu peux ouvrir une autre issue.

@rezemika
Copy link
Contributor

rezemika commented Aug 3, 2017

Oui, ceux-là, je les avais bien repérés, mais je parlais des tableaux qu'on peut voir sur d'autres pages (surtout celles des validateurs il me semble).

Ok pour l'issue, je ferais ça dès demain.

rezemika added a commit to rezemika/zds-site that referenced this issue Aug 4, 2017
rezemika added a commit to rezemika/zds-site that referenced this issue Aug 5, 2017
vhf pushed a commit that referenced this issue Aug 6, 2017
* Corrige l'affichage du tableau sur les petits écrans (#4427)

* Corrige un conflit

* Ajoute des classes 'wide' sur certaines colonnes des tableaux
@artragis
Copy link
Member

@vhf @rezemika ça semble fixé, non?

@gllmc
Copy link
Member

gllmc commented Aug 12, 2017

Je confirme que c'est fixé par #4446. :)

@gllmc gllmc closed this as completed Aug 12, 2017
sandhose pushed a commit that referenced this issue Sep 18, 2017
* Corrige l'affichage du tableau sur les petits écrans (#4427)

* Corrige un conflit

* Ajoute des classes 'wide' sur certaines colonnes des tableaux
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

6 participants