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.

Sign up for GitHub

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

Jump to bottom

Corrige l'affichage du tableau sur les petits écrans (#4427) #4446

Merged
merged 3 commits into from
Aug 6, 2017
Merged

Corrige l'affichage du tableau sur les petits écrans (#4427) #4446

merged 3 commits into from
Aug 6, 2017

Conversation

rezemika
Copy link
Contributor

@rezemika rezemika commented Aug 4, 2017

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #4427

QA

  • Éditer un des messages par défaut dans les fixtures

  • Aller sur la page de l'historique d'édition du message (http://localhost:8000/pages/historique-editions/5/ pour le premier)

  • Passer en mode "vue adaptative" et vérifier l'affichage du tableau pour des écrans de petites tailles

  • Code relu et approuvé.

  • Ça fonctionne !

PS : Y a-t-il d'autres pages concernées par ce problème ?

@coveralls
Copy link

coveralls commented Aug 4, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 89.278% when pulling 1e6a93b on rezemika:fix-4427 into fcf26ba on zestedesavoir:dev.

@vhf vhf added C-Front Concerne l'interface du site QA svp labels Aug 4, 2017
@vhf
Copy link
Contributor

vhf commented Aug 5, 2017

@rezemika tu peux faire pareil sur la page d'affichage des alertes stp ?

2017-08-02 11 33 17

@coveralls
Copy link

coveralls commented Aug 5, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 89.273% when pulling d723821 on rezemika:fix-4427 into 233d031 on zestedesavoir:dev.

motet-a
motet-a reviewed Aug 6, 2017
View reviewed changes
Copy link
Contributor

@motet-a motet-a left a comment •
edited
Loading

Choose a reason for hiding this comment

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

image

image

C’est assez écrasé. À mon avis, il y a deux solutions :

  • Autoriser le défilement horizontal sur le tableau avec un overflow-x: auto;. C’est moche mais ça se fait en deux secondes.

  • Dégager le tableau et le remplacer par quelque chose de responsive mais c’est plus long.

@vhf
Copy link
Contributor

vhf commented Aug 6, 2017

Pour les éditions je propose de mettre Date en wide.

Pour les alertes je propose de mettre Type et Date en wide.

@coveralls
Copy link

coveralls commented Aug 6, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 89.273% when pulling 9736b65 on rezemika:fix-4427 into 233d031 on zestedesavoir:dev.

@vhf vhf merged commit 080f6ef into zestedesavoir:dev Aug 6, 2017
@vhf vhf modified the milestone: Version de développement Aug 6, 2017
@rezemika rezemika deleted the fix-4427 branch August 6, 2017 15:12
@gllmc gllmc mentioned this pull request Aug 12, 2017
Closed
@gllmc gllmc removed the QA svp label Aug 12, 2017
sandhose pushed a commit that referenced this pull request Sep 18, 2017
@rezemika @sandhose
Corrige l'affichage du tableau sur les petits écrans (#4427) (#4446)
340d606 
* 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
Reviewers

@motet-a motet-a motet-a left review comments

Assignees
No one assigned
Labels
C-Front Concerne l'interface du site
Projects
None yet
Milestone
v25
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@rezemika @coveralls @vhf @motet-a @gllmc