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

[IMPORT] [FRONTEND] [PROPOSITION] Modification du layout du rapport d'import #3041

Conversation

edelclaux
Copy link
Contributor

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:

  • marge et padding un peu partout
  • containers de section de material à boostrap
  • le header (précédent, titre, status) sur une seule ligne
  • taille des polices, des icones, des titres ajustées.
  • équilibre horizontal contenu / bouton en faveur du contenu

Rapport d'import
image

Liste des imports
image

@edelclaux edelclaux changed the base branch from master to feat/import May 13, 2024 09:34
@edelclaux edelclaux changed the title [IMPORT] [FRONTEND] Feat import/add sinp standard in import report [IMPORT] [FRONTEND] Modification du layout du rapport d'import May 14, 2024
@edelclaux edelclaux force-pushed the feat-import/add-sinp-standard-in-import-report branch from 139a8e7 to 5ceb7c3 Compare May 14, 2024 16:20
@edelclaux edelclaux marked this pull request as ready for review May 15, 2024 07:40
@edelclaux edelclaux changed the title [IMPORT] [FRONTEND] Modification du layout du rapport d'import [IMPORT] [FRONTEND] [PROPOSITION] Modification du layout du rapport d'import May 15, 2024
@edelclaux edelclaux marked this pull request as draft May 16, 2024 16:23
@edelclaux edelclaux marked this pull request as ready for review May 29, 2024 14:53
@edelclaux edelclaux force-pushed the feat-import/add-sinp-standard-in-import-report branch from 5829635 to e0fac88 Compare May 29, 2024 15:51
@jacquesfize jacquesfize force-pushed the feat/import branch 2 times, most recently from 58fdc22 to fd291fc Compare July 25, 2024 14:57
@bouttier bouttier force-pushed the feat/import branch 2 times, most recently from 22e3600 to 3cc485f Compare August 14, 2024 13:42
@bouttier bouttier force-pushed the feat-import/add-sinp-standard-in-import-report branch from e0fac88 to 8b7978a Compare August 14, 2024 17:05
@bouttier bouttier self-requested a review August 14, 2024 17:06
Copy link
Contributor

@bouttier bouttier left a 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.

@edelclaux
Copy link
Contributor Author

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.

Copy link

codecov bot commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.54%. Comparing base (c7bb770) to head (39b109a).
Report is 1 commits behind head on feat/import.

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           
Flag Coverage Δ
pytest 83.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Pierre-Narcisi Pierre-Narcisi force-pushed the feat-import/add-sinp-standard-in-import-report branch from c124064 to 39b109a Compare October 16, 2024 14:55
@jacquesfize jacquesfize merged commit c98a1cc into PnX-SI:feat/import Oct 17, 2024
7 checks passed
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.

3 participants