Feat/377 improve mobile asset select #389
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
# Conflicts: # package-lock.json # package.json
|
De code ziet er erg goed uit. Mooi dat nu de MapDialog Hooks zijn geïntroduceerd. Dat er een mobiel component is en een desktop component. Ik zie wel dat je de gehele useHooks library nu gebruikt binnen het project, is het een idee om enkel de code van de hook die wij gebruiken op te nemen in het project (gewoon in onze codebase) en niet de gehele library te gebruiken? |
Soms valt in de lijstweergave van assets optie de knop van het volledige scherm af, dan staat deze er half op. Is dit de bedoeling? |
|
Moet iemand hier nog naar kijken om alles 100% door te testen op alle a11y punten? |
|
Bij een pin op de kaart in de mobiele weergave veranderd de hoogte van de map constant even (soort flikkering) doordat dan weer die zoom melding wordt laten zien. Is het een idee om hier een of andere timeOut of animatie om heen te bouwen die gecancelled wordt als het zoomniveau kort daarna weer op een minimaal niveau is? |
|
Voor de rest kom ik geen gekke dingen tegen. |
voor #377