-
Notifications
You must be signed in to change notification settings - Fork 103
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
[IMPORT] [FRONTEND] [PROPOSITION] Modification du layout du rapport d'import #3041
[IMPORT] [FRONTEND] [PROPOSITION] Modification du layout du rapport d'import #3041
Conversation
139a8e7
to
5ceb7c3
Compare
5829635
to
e0fac88
Compare
58fdc22
to
fd291fc
Compare
22e3600
to
3cc485f
Compare
e0fac88
to
8b7978a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR rebasé.
J’ai un scroll horizontal moche et peu utilisable sous firefox sur la liste des imports (probablement déjà avant cette PR mais ça peut être l’occasion de corriger).
Sinon je suis plutôt pour ce design.
Tu peux joindre une capture d'écran pour illustrer le scroll horizontal ? Avec éventuellement ta résolution d'écran ?) Si c'est un scroll ciblé sur le ngx-datatable, je serai pour le traiter dans une issue séparée --> https://github.com/orgs/PnX-SI/projects/13/views/15?filterQuery=-status%3AFini+-is%3Apr+datatable&pane=issue&itemId=64947167 Sinon, carrément, je peux regarder. |
dcdde5a
to
d64cff9
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/import #3041 +/- ##
============================================
Coverage 83.54% 83.54%
============================================
Files 121 121
Lines 9559 9559
============================================
Hits 7986 7986
Misses 1573 1573
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
c124064
to
39b109a
Compare
Cette PR porte sur le layout
Ceci est une proposition.
J'ai fait un essai de refacto de la page importe report, pour qu'elle soit plus cohérente avec la liste des imports.
Modifications:
Rapport d'import
Liste des imports