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

[FEATURE] améliore l'affichage de PixStructureSwitcher (pix-15674) #788

Merged
merged 1 commit into from
Dec 13, 2024
Merged

[FEATURE] améliore l'affichage de PixStructureSwitcher (pix-15674) #788

merged 1 commit into from
Dec 13, 2024

Conversation

lionelB
Copy link
Member

@lionelB lionelB commented Dec 11, 2024 •
edited
Loading

🎄 Problème

  • Le composant de choix de structure peut faire penser qu’il est dans la zone contenu au lieu de la navigation
  • Pour certains utilisateurs la zone peut être trop étroite pour lire correctement le nom du centre/orga

🎁 Proposition

Rapprocher le dropdown de la navigation
Supprimer l'affichage sur une ligne avec l'ellipse

🌟 Remarques

RAS

🎅 Pour tester

Afficher la story du PixStructureSwitcher

@lionelB lionelB self-assigned this Dec 11, 2024
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr788.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr788/environment

@lionelB lionelB force-pushed the PIX-15674/update-pix-navigation branch 2 times, most recently from 3cd388a to 67b930b Compare December 11, 2024 13:01
@AndreiaPena
Copy link
Member

AndreiaPena commented Dec 11, 2024 •
edited
Loading

Niquel pour l'affichage en 2 lignes

Capture d’écran 2024-12-11 à 14 16 02

En revanche quand je clique dessus, on a ce résultat

Capture d’écran 2024-12-11 à 14 18 42

Même résultat sur la version mobile

Capture d’écran 2024-12-11 à 14 41 33

@AndreiaPena
Copy link
Member

AndreiaPena commented Dec 11, 2024 •
edited
Loading

HS à ce que fait la PR : Une question plutôt pour le design. (@pierrepougetpix @QuentinChapelain-ui )

J'avais pas fait gaffe avant de tester là qu'une couleur est appliquée au texte dans la liste.
(quand on parcours la liste au clavier pour être plus précise)
Si le bleu foncé marche plutôt bien avec le bleu d'Orga, le contraste avec le vert de Certif et celui de App semble poser problème.

Je propose de créer un ticket pour modifier la couleur selon les variants. Qu'en pensez-vous ?

La version Orga

Enregistrement.de.l.ecran.2024-12-11.a.14.28.46.mov

La version Certif

Enregistrement.de.l.ecran.2024-12-11.a.14.28.59.mov

La version App

Enregistrement.de.l.ecran.2024-12-11.a.14.37.29.mov

@lionelB lionelB force-pushed the PIX-15674/update-pix-navigation branch from 67b930b to 0d5a232 Compare December 11, 2024 14:49
@AndreiaPena
Copy link
Member

AndreiaPena commented Dec 11, 2024 •
edited
Loading

La liste fusionne avec le menu :o

Capture d’écran 2024-12-11 à 18 10 02

Je note aussi qu'on perd du padding en mobile
(mais après ça peut se régler coté app)

Capture d’écran 2024-12-11 à 18 12 06 Capture d’écran 2024-12-11 à 18 12 37

@lionelB lionelB force-pushed the PIX-15674/update-pix-navigation branch from 0d5a232 to e23e3cf Compare December 12, 2024 13:06
@lionelB
Copy link
Member Author

lionelB commented Dec 12, 2024

@AndreiaPena merci c'est corrigé

@xav-car
Copy link
Contributor

xav-car commented Dec 12, 2024

La liste fusionne avec le menu :o

Souhait du Design 😅

Je note aussi qu'on perd du padding en mobile
(mais après ça peut se régler coté app)

Corrigé !

@AndreiaPena
Copy link
Member

La liste fusionne avec le menu :o

Souhait du Design 😅

Ah ! D'accord 👀

@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-15674/update-pix-navigation branch from e23e3cf to 2132fc8 Compare December 13, 2024 10:08
@pix-service-auto-merge pix-service-auto-merge merged commit d907cb7 into dev Dec 13, 2024
3 of 5 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-15674/update-pix-navigation branch December 13, 2024 10:09
pix-service-auto-merge pushed a commit that referenced this pull request Dec 13, 2024
@semantic-release-bot
chore(release): 51.5.0 [skip ci]
6bd4414 
# [51.5.0](v51.4.0...v51.5.0) (2024-12-13)

### 🚀 Amélioration

- [#788](#788) améliore  l'affichage de PixStructureSwitcher (pix-15674)
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 51.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

lionelB pushed a commit that referenced this pull request Dec 13, 2024
@semantic-release-bot @lionelB
chore(release): 51.5.0 [skip ci]
7cd55f8 
# [51.5.0](v51.4.0...v51.5.0) (2024-12-13)

### 🚀 Amélioration

- [#788](#788) améliore  l'affichage de PixStructureSwitcher (pix-15674)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AndreiaPena AndreiaPena AndreiaPena approved these changes

@xav-car xav-car xav-car approved these changes

Assignees

@lionelB lionelB

Labels
Design review OK Func Review OK released 🚀 Ready to Merge Tech review OK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@lionelB @pix-bot-github @AndreiaPena @xav-car @pix-service-auto-merge @pierrepougetpix