Feat/home big title and search #556
Conversation
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for meteo-france ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Il me semble que le dropdown est graphiquement cassé sur |
Le |
at-github
force-pushed
the
feat/home-big-title-and-search
branch
from
fb477c3 to
afc24ea
Compare
|
Le dernier commit fix la régression du menu déroulant de la recherche post mise à jour des composants |
at-github
force-pushed
the
feat/home-big-title-and-search
branch
from
afc24ea to
1ff5788
Compare
|
Fix du dropdown récupéré. |
There was a problem hiding this comment.
- Je ne vois pas le bleu sur le titre.
- La barre de recherche est plus large que sur la maquette (écran 1440px)
- Je trouve le comportement du double placeholder étonnant : afficher le hint seulement au focus — de plus, pas cohérent avec le header search (il y a peut-être des discussions là-dessus que j'ai raté). Je mettrais un seul placeholder qui est le hint.
at-github
force-pushed
the
feat/home-big-title-and-search
branch
from
1ff5788 to
8d2809f
Compare
| @@ -69,7 +72,7 @@ const dropdownLabel = (text: string) => { | |||
| :options="dropdown" | |||
| label="text" | |||
| track-by="route" | |||
| placeholder="" | |||
| :placeholder="placeholder" | |||
There was a problem hiding this comment.
ce n'est peut-être pas la peine d'avoir une valeur ici vu qu'on override avec le slot placeholder plus bas non ?
There was a problem hiding this comment.
D'après ma compréhension et mes tests :
- Ligne 79 : ce placeholder s'affiche après focus
- Ligne : 103 : ce placeholder s'affiche avant focus
|
Pull request move to #574 |
Fix ecolabdata/ecospheres#392