SOS-RS / frontend

Auxílio RS: Projetos de Resposta a Emergências por Chuvas e Alagamentos
https://sos-rs.com
MIT License
596 stars 320 forks source link

Adjust viewport height for mobile devices on Update Shelter page #232

Closed felipemrvieira closed 5 months ago

felipemrvieira commented 6 months ago

ISSUE 225

felipemrvieira commented 6 months ago

192 168 0 12_5173_abrigo_3a5e8ab5-5c48-48f3-98f1-f7c9c5b26ddb_atualizar(Pixel 7) 192 168 0 12_5173_abrigo_3a5e8ab5-5c48-48f3-98f1-f7c9c5b26ddb_atualizar(iPhone 14 Pro Max) Screenshot from 2024-05-17 13-14-55 pocox3

allangalera commented 6 months ago

Postei na issue original. Acredito que podemos resolver isso usando apenas css -> https://caniuse.com/viewport-unit-variants

vinny-silveira commented 5 months ago

Postei na issue original. Acredito que podemos resolver isso usando apenas css -> https://caniuse.com/viewport-unit-variants

Bem mais simples mesmo essa abordagem @allangalera, 92.34% dos browsers e versões já suportam essa unidade:

image

@felipemrvieira, como temos um suporte para mais de 90%, considere o uso do dvh por gentileza.

felipemrvieira commented 5 months ago

Postei na issue original. Acredito que podemos resolver isso usando apenas css -> https://caniuse.com/viewport-unit-variants

Bem mais simples mesmo essa abordagem @allangalera, 92.34% dos browsers e versões já suportam essa unidade:

image

@felipemrvieira, como temos um suporte para mais de 90%, considere o uso do dvh por gentileza.

Acabei de atualizar o PR. Valeu pelo feedback!

vinny-silveira commented 5 months ago

@SOS-RS/frontenders Preciso de mais um review aqui.

felipemrvieira commented 5 months ago

Acredito que está pronto para merge @vinny-silveira @andradeviniicius @larissapissurno

larissapissurno commented 5 months ago

mergeado @felipemrvieira