fix(AVO-4043): replace avo2 components tooltip with local version#345
fix(AVO-4043): replace avo2 components tooltip with local version#345bertyhell wants to merge 1 commit into
Conversation
https://meemoo.atlassian.net/browse/AVO-4043 using @meemoo/react-components tooltip
| @@ -1,3 +1,4 @@ | |||
| import { Tooltip, TooltipContent, TooltipTrigger } from '@meemoo/react-components'; | |||
There was a problem hiding this comment.
Willen we deze wijzigingen zo doorduwen?
|
|
||
| .p-content-page-publish-modal { | ||
| .c-tooltip-component { | ||
| background-color: #edeff2; |
There was a problem hiding this comment.
Hoort deze styling hier wel thuis?
There was a problem hiding this comment.
eigenlijk niet, maar de Tooltip component zit in de react-component en moet eigenlijk gestyled worden in de client
maar de admin-core zit daar tussenin
admin-core moet eigenlijk niet gestyled zijn, maar de styling van die arrow triangle kan enkel via properties op de tooltip component
dus deze tooltip is wel styled
de enige andere optie die ik zie is die styles laten setten in de admin-core config en ze dan zo doorgeven aan de tooltip component. Maar dat leek me wat overkill voor die ene tooltip die we in de admin-core schermen hebben zitten
| ['bio', tText('admin/users/views/user-detail___bio')], | ||
| ['stamboek', tText('admin/users/views/user-detail___stamboek-nummer')], | ||
| ['email', tText('admin/users/views/user-detail___primair-email-adres')], | ||
| [ |
There was a problem hiding this comment.
deze is al verwijderd in het verleden, maar blijkbaar bestond deze entry nog, maar dat is niet meer geldig volgens typescript typechecking
alias en alternative_email is gewist geweest aangezien die velden niet gebruikt worden en ze vaak voor problemen zorgen met max-length issues bij registratie
2 participants
https://meemoo.atlassian.net/browse/AVO-4043
part of:
using @meemoo/react-components tooltip