-
Notifications
You must be signed in to change notification settings - Fork 15
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
Refactor du Summary/table view #2633
Comments
Only the section regarding the tabs will be deployed and another story will be created for the portion with the buttons. |
Tested successfully on Chrome and Firefox |
@francisl Salut Francis, J'ai ajouté les petits points à corriger dans la description en haut avec un imprime écran. Laisse-moi savoir si tu as des questions ! |
@luclemo about the second point. It's not from a font icon library, default antd or svg. they are not customizable. @denis-crsj please create a new design ticket to provide the proper format, so we can theme the icons. |
@francisl Ok pas de soucis, si on les embed directement plutôt q'un img source ça devrait être plus flexible. J'ai les SVG si @denis-crsj veut les ajouter au nouveau ticket. |
@francisl @denis-crsj Désolé j'avais pas vue ça passer. Si tu a déjas apporté la correction tant mieux, sinon tu peux laisser faire ça. |
@francisl voici les icônes en fichiers svg individuels et aussi un sprite suivant ce que tu préfère pour les intégrer: EDIT: New zip just with individual icons: kf-icons-single.zip |
Maquettes 👇
https://www.figma.com/file/xwFNKpa1zXoCg1xaJhAvKn/KF---Saved-Sets?node-id=18%3A56&viewport=1499%2C774%2C0.6173954606056213
Defects
ant-btn-link
smallThe text was updated successfully, but these errors were encountered: