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

[FEATURE] Ajout d'une bordure à gauche des feedbacks (PIX-13261) #9458

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

dlahaye
Copy link
Contributor

@dlahaye dlahaye commented Jul 5, 2024

🦄 Problème

Aujourd'hui, les feedbacks Modulix nécessitent une révision de leur UX pour qu'elles soient plus facilement identifiables comme un bloc à part entière.

🤖 Proposition

Ajouter une bordure à gauche des feedbacks en suivant la maquette de Quentin.

🌈 Remarques

J'ai fait différemment de ce qui a été implémenté sur Pix Admin afin de ne pas casser le CSS relatif à la contribution (conserver le BEM de .feedback__state).

💯 Pour tester

  1. Se rendre sur le didacticiel
  2. Répondre à des modalités
  3. S'assurer que le rendu respecte les maquettes

@dlahaye dlahaye self-assigned this Jul 5, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@dlahaye dlahaye marked this pull request as ready for review July 5, 2024 14:55
Copy link
Contributor

@AnaisAllamand AnaisAllamand left a comment

Choose a reason for hiding this comment

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

OK desktop et mobile !

@QuentinChapelain-ui
Copy link

Coucou, pourrait-on ajouter 6 px de padding entre la bordure et le texte. En fait j'ai l'impression qu'il ne prends pas en compte la largeur de la bordure mais commence à -6px donc passer de 3X de padding à 4X.
Capture d’écran 2024-07-08 à 16 04 49

Co-authored-by: Rébecca Kaci <82950611+reibecca@users.noreply.github.com>
@dlahaye dlahaye force-pushed the PIX-13261-add-border-to-feedbacks branch from bf125c3 to bd8eda5 Compare July 8, 2024 14:26
@pix-service-auto-merge pix-service-auto-merge merged commit 0f2fa44 into dev Jul 8, 2024
7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-13261-add-border-to-feedbacks branch July 8, 2024 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants