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] Mise à jour de l'icône tabletOff (PIX-15426) #769

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

mcampourcy
Copy link
Contributor

🎄 Problème

Sur les nouvelles icônes, pour les tablettes 'off' (barrées), on ne voit pas la différence entre téléphone et tablette

Capture d’écran 2024-11-25 à 15 05 38 Capture d’écran 2024-11-25 à 15 05 28 1

🎁 Proposition

Élargir l'icône tablette

Capture d’écran 2024-11-25 à 15 00 09 Capture d’écran 2024-11-25 à 14 59 53 1

🌟 Remarques

Validé côté design

🎅 Pour tester

Vérifier sur la liste des icônes que tabletOff est OK

@pix-bot-github
Copy link

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

@xav-car
Copy link
Contributor

xav-car commented Nov 25, 2024

question Je dirais qu'il manque un petit tiret blanc dans le tabletOff_plain si on souhaite rester cohérent avec les autres tablet.

@xav-car
Copy link
Contributor

xav-car commented Nov 27, 2024

Vu avec le Design, on fera le correctif plus tard. c'est pas trop trop grave. bisous

@pix-service-auto-merge pix-service-auto-merge merged commit 25e83e1 into dev Nov 27, 2024
4 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15426-add-tablet-off-icons branch November 27, 2024 09:59
pix-service-auto-merge pushed a commit that referenced this pull request Nov 27, 2024
# [49.1.0](v49.0.0...v49.1.0) (2024-11-27)

### 🚀 Amélioration

- [#769](#769) Mise à jour de l'icône tabletOff (PIX-15426)
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 49.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

5 participants