Alertes – Corrige la sélection de façade lorsque l'on ouvre une alerte à partir de la fiche navire #3631
Conversation
ivangabriele
force-pushed
the
ivan/fix-alerts-opening-from-main-window-vessel-card
branch
from
56432fd
to
5ff6f08
Compare
|
| @@ -117,16 +117,19 @@ const Layers = styled.div<{ | |||
| max-height: calc(100vh - ${p => (p.$hasHealthcheckTextWarning ? '210px' : '160px')}); | |||
| ` | |||
|
|
|||
| const SidebarLayersButton = styled(MapButton)` | |||
There was a problem hiding this comment.
Pourquoi tu n'aime pas tout mettre dans MapButton ? ça permet d'avoir dans un seul endroit :
margin-top: ${p => (p.$hasHealthcheckTextWarning ? 50 : 0)}px;
C'est plus facile à refacto non ?
There was a problem hiding this comment.
C'est parce que j'ai galéré un poil à fixer ce soucis de <button> dans un <button> et c'était la manière de faire que j'ai trouvé la plus simple. Si je passe simplement un span ou un div ça casse tout le positionning.
Mais je suis d'accord que c'est mieux d'avoir un bouton commun. Par contre je te propose de plutôt faire ça dans #3131 si ça te va ? (j'ai presque fini de rebase cette vieille PR pendant qu'on faisait le weekly)
There was a problem hiding this comment.
Ah oui d'acc on pourra voir ça dans la #3131
ivangabriele
deleted the
ivan/fix-alerts-opening-from-main-window-vessel-card
branch
Linked issues