Develop rewrite ffe tables #2142
Conversation
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
from
2fd93c9 to
4b6a09e
Compare
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
28 times, most recently
from
c32b6df to
b64960c
Compare
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2142.westeurope.2.azurestaticapps.net |
packages/ffe-tables/less/tables.less
Outdated
| } | ||
| } | ||
|
|
||
| .ffe-table__caption { |
There was a problem hiding this comment.
Skal det ikke heller skrives &__caption, inne i ffe-table?
There was a problem hiding this comment.
Gjelder alt i denne fila
There was a problem hiding this comment.
Jag liker ikke det selv, men hvis dere båda vill det skall vi gjøra det. Jag synes all den nestningen blir svært oøversiktlig også ær det lett og lage mye mer spesifitet en man trenger. Jag liker noen nestning men ikke less nestning før att slå sammen klassenavn :)
There was a problem hiding this comment.
Virker som det varierer litt hvordan det er løst i forskjellige komponenter, så jeg vet ikke helt hva fasit egentlig er. Men for meg gjør nesting koden mer oversiktlig, ikke mindre.
There was a problem hiding this comment.
Ok jag kan gjøre det hvis vi skall vare kvar på less. Det som er oversiktligt før meg er hvis jag ser __ett-eller-annet så må jag setta ihop hela klassenavnet i hode mitt. Det går ju bra når er lite kode. Jag har ikke heller fasit alltså. Bare vad jag føler runt det.
.min {
&___ja {
&-vart {
&-var-jeg-nå {.....}
}
}
}
There was a problem hiding this comment.
Virker som det varierer litt hvordan det er løst i forskjellige komponenter, så jeg vet ikke helt hva fasit egentlig er. Men for meg gjør nesting koden mer oversiktlig, ikke mindre.
Nu har jag iaf lessifisert stylinga :)
Tusen takk... bare bra du spør/sier ifra |
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
from
27abe64 to
0c10dbd
Compare
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2142.westeurope.2.azurestaticapps.net |
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
from
0c10dbd to
16a6ddb
Compare
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2142.westeurope.2.azurestaticapps.net |
Dette finnes ikke i Figma, så kan kanskje være lurt å ta en synk med design om mønsteret før merge? Savner transition når pila skifter retning, men det blir kanskje vanskelig å få til hvis det er 3 forskjellige ikoner. |
Ja det er akkurat det med 3 olika ikoner. Det er ju ingenting som kan roteres. |
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
from
16a6ddb to
452faa5
Compare
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2142.westeurope.2.azurestaticapps.net |
BREAKING CHANGE: complete rewrite of component
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
from
452faa5 to
0986ff3
Compare
BREAKING CHANGE: Compent API is new
pethel
force-pushed
the
develop_rewrite-ffe-tables
branch
from
0986ff3 to
996104d
Compare
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2142.westeurope.2.azurestaticapps.net |
1 similar comment
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2142.westeurope.2.azurestaticapps.net |
Jag har sjekket av med design nå @antidecaf 🍰
|
Da har jag gjort ett førsøk på att skriva om Table slik att den forhopningsvis er mindre komplisert og mer universielt utrformad.
Jag begynner veldig minimalistiskt. Ser gamla tabellen hade en responsiv variant som ikke var dolumentert og ser ikke spår av den i figma. Annars mangler også mulighet att skjula kolumner avhenge av skjermbredde. Er det noe vi trenger?
Ser varken aksel eller designsystemet,no ser ut att ha noen av disse.
Sorteringen har også fått ett tredje ikon før staten "osortert". Tror det vill hjelpa på UU det med.