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

feat: Add red color for puik-card (needed for the tech-console) #256

Closed
wants to merge 4 commits into from

Conversation

guillaume60240
Copy link
Collaborator

❓ Types of changes

Add the red color on puik-card

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • 📦 New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes
  • The component exists on old Prestashop UIKit and my pull request on migrating documentation is accepted.

@guillaume60240 guillaume60240 self-assigned this Nov 23, 2023
@mattgoud mattgoud self-requested a review November 24, 2023 12:35
@mattgoud
Copy link
Collaborator

@FrancoisQtr, @guillaume60240 a crée un variant "red" pour le composant card pour les besoins de CloudSync. Peux-tu faire la review UI pour sa PR ?

@mattgoud mattgoud requested a review from FrancoisQtr November 24, 2023 12:40
@kktos
Copy link

kktos commented Nov 24, 2023

@FrancoisQtr, @guillaume60240 a crée un variant "red" pour le composant card pour les besoins de CloudSync. Peux-tu faire la review UI pour sa PR ?

Cela me fait me demander pourquoi une variante "red" de la card......
Se pourrait il que ce soit pour indiquer une chose potentiellement grave ?
Cela serait alors une alerte...
Et il y a déjà un composant pour cela.

@guillaume60240
Copy link
Collaborator Author

@FrancoisQtr, @guillaume60240 a crée un variant "red" pour le composant card pour les besoins de CloudSync. Peux-tu faire la review UI pour sa PR ?

Cela me fait me demander pourquoi une variante "red" de la card...... Se pourrait il que ce soit pour indiquer une chose potentiellement grave ? Cela serait alors une alerte... Et il y a déjà un composant pour cela.

Hello @FrancoisQtr
Pour moi le rôle d'une alerte est l'affichage d'un message ponctuel alors que nous sommes dans le context d'un widget qui affiche l'état des composants CloudSync ce qui fait une grosse différence de sémantique à mon sens.
De plus le design d'un alert n'est pas le même que celui des cards.
On a bien une couleur green alors que le composant alert propose déjà cette couleur

J'espère que ces informations te permettront de comprendre plus ce besoin

@cnavarro-prestashop
Copy link
Contributor

@guillaume60240 / @kktos / @mattgoud / @FrancoisQtr : on en est où de la décision design sur cette PR svp ?

@mattgoud
Copy link
Collaborator

ultimately no red variant for card

@mattgoud mattgoud closed this Jan 17, 2024
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.

4 participants