RO-2773: Endringer knytta til maks bredde på sider#731
Conversation
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-731.westeurope.azurestaticapps.net |
1 similar comment
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-731.westeurope.azurestaticapps.net |
jorgkv
left a comment
jorgkv
left a comment
There was a problem hiding this comment.
Har du vist dette til Maren? Jeg syns dette fungerer bra, men maks bredde i skjema er for bred syns jeg. Maren sa hun hadde en bredde som egnet seg til skjema? Tipper kanskje vi bør ha en standard maks bredde og en egen maks bredde for skjema.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-731.westeurope.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-731.westeurope.azurestaticapps.net |
jorgkv
left a comment
jorgkv
left a comment
There was a problem hiding this comment.
I stedet for å sette maks bredde på alle subkomponentene inne på my-observations.page, er det ikke enklere å bare sette maks bredde direkte på my-observations.page ?
Jeg tenker da på
sent-list, draft-list, empty-state, error-state, osv.
src/app/components/observation/observation-skeleton/observation-skeleton.component.scss
Show resolved
Hide resolved
| color: inherit; | ||
| } | ||
|
|
||
| ion-content { |
There was a problem hiding this comment.
Jeg tror det kan være lurt å ikke sette maks bredde rett på ion-content, men bruke en ekstra div med maks-bredde inni ion-content. Det virker som det er noen animasjoner som har med ion-content å gjøre som nå ser litt rare ut.
There was a problem hiding this comment.
setter max bredde på ion-grid her.
| } | ||
| } | ||
|
|
||
| ion-content { |
There was a problem hiding this comment.
I stedet for å sette denne cssen rett på ion-content så tror jeg det er bedre å lage en subkomponent med maks bredde. Router-animasjonene ser litt rare ut etter denne endringen.
There was a problem hiding this comment.
legger til <main> element
hvis jeg setter maks bredde på my observations.page får jeg dette vi burde egentlig ha noe element som skal inneholde alt på siden, og blir enklere å sette layout, men siden vi ikke har den, så må man nesten gjøre en slik hokus pokus.Det kan hjelpe med en hoved div wrapper da. Jeg legger til |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-731.westeurope.azurestaticapps.net |
|
Jeg vil bare si at jeg vil gjerne jobbe litt mer med layout i appen i fremtiden, sørge for riktige landmarks og håpentligvis gjøre selve layouten litt enklere å jobbe med. Layout i selve appen virker enkelt fordi vi har fast bredde på de fleste sider, og har ikke så mange sider heller. Tror vi kan sette hoved bredde på alle page-ne mer globalt hvis vi f.eks. tar i bruk |
jorgkv
left a comment
jorgkv
left a comment
There was a problem hiding this comment.
Veldig bra, nå så animasjonene inn i mine observasjoner, min side osv mye bedre ut. Bra jobba! Se kommentaren, men bare ta inn PR etter du evt har fiksa den.
| @@ -1,4 +1,3 @@ | |||
| <!-- A list of observations sent to server --> | |||
There was a problem hiding this comment.
Denne kommentaren er kanskje fjerna uten grunn?
2 participants
Nå bruker vi kun en css property for å sette maks bredde på store skjermer.
Som beskrevet i kommentaren i global.scss, det hadde vært fint om vi kunne sette layouten et sted, ikke i hver enkelt komponent
men dette kan diskuteres senere.
Jeg har en stash lokalt hvor jeg også fikser maks bredde i mine observasjoner. Jeg kan gjerne ta den med, men den der krevet litt mer omskriving i de enkelte komponenter knyttet til mine observasjoner.