Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ffe-header): remove button dependencies #2317

Merged
merged 1 commit into from
Sep 24, 2024
Merged

Conversation

pethel
Copy link
Contributor

@pethel pethel commented Sep 24, 2024

Det var overaskande att dette skulle havne på vårt bord... veldig...

Fjerner alle avheningheter til buttons

@pethel pethel force-pushed the develop_fix-header branch 2 times, most recently from fabf75c to 956936c Compare September 24, 2024 11:14
@pethel pethel marked this pull request as ready for review September 24, 2024 11:14
@pethel pethel requested a review from a team as a code owner September 24, 2024 11:14
@pethel pethel force-pushed the develop_fix-header branch 3 times, most recently from 3a6c7bf to a749ca9 Compare September 24, 2024 11:22
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2317.westeurope.2.azurestaticapps.net

1 similar comment
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2317.westeurope.2.azurestaticapps.net

@tuva-odegard
Copy link
Contributor

tuva-odegard commented Sep 24, 2024

Nå har knappen fått en underline, er det meningen?
image

Den skall ju vare lik men gjentar ju stylinga. Kan sikkert justeras. Det er på hover kanskej bare. Jag fikser

@tuva-odegard
Copy link
Contributor

tuva-odegard commented Sep 24, 2024

Og den har mistet onclick-animasjonen

@tuva-odegard
Copy link
Contributor

Og litt annen farge i darkmode - vet ikke om det er bedre kanskje, men annerledes i alle fall, ikke sikker på om det var meningen?
image
image

@pethel
Copy link
Contributor Author

pethel commented Sep 24, 2024

Og den har mistet onclick-animasjonen

Kan vi leva med det og kanskje jsutera hvis det trengs slik att vi ikke trenger og kopiera hela buttons før en logut knapp? :)

Vi vill ju aller helst ikke ens løsa det slikt. Blir så kjipt att ha hele ffe-buttons 2 steder før oss liksom

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2317.westeurope.2.azurestaticapps.net

@pethel
Copy link
Contributor Author

pethel commented Sep 24, 2024

Og litt annen farge i darkmode - vet ikke om det er bedre kanskje, men annerledes i alle fall, ikke sikker på om det var meningen? image image

Jag kopierte fra tertiary-button, som denne gentligen extenda fra. Vet ikek vorfor den er mørkare egentligen
image

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2317.westeurope.2.azurestaticapps.net

@antidecaf
Copy link
Contributor

Og den har mistet onclick-animasjonen

Kan vi leva med det og kanskje jsutera hvis det trengs slik att vi ikke trenger og kopiera hela buttons før en logut knapp? :)

Vi vill ju aller helst ikke ens løsa det slikt. Blir så kjipt att ha hele ffe-buttons 2 steder før oss liksom

Det er bare en :active state med transform, så burde ikke være så altfor komplisert å legge tilbake?

Det er denne som mangler sammenlignet med prod:

image

Gjelder både navnet og Logg ut-knappen.

@pethel
Copy link
Contributor Author

pethel commented Sep 24, 2024

Og den har mistet onclick-animasjonen

Kan vi leva med det og kanskje jsutera hvis det trengs slik att vi ikke trenger og kopiera hela buttons før en logut knapp? :)
Vi vill ju aller helst ikke ens løsa det slikt. Blir så kjipt att ha hele ffe-buttons 2 steder før oss liksom

Det er bare en :active state med transform, så burde ikke være så altfor komplisert å legge tilbake?

Det er denne som mangler sammenlignet med prod:

image

Gjelder både navnet og Logg ut-knappen.

Sånn. Lagt på det også.

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2317.westeurope.2.azurestaticapps.net

@pethel pethel merged commit b6aae7c into develop Sep 24, 2024
3 checks passed
@pethel pethel deleted the develop_fix-header branch September 24, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants