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

feat(ffe-header): legge til semantiske farger #2567

Closed
wants to merge 1 commit into from

Conversation

hagenek
Copy link
Contributor

@hagenek hagenek commented Feb 21, 2025

Beskrivelse

Motivasjon og kontekst

Testing

@hagenek hagenek requested a review from a team as a code owner February 21, 2025 13:54
@hagenek hagenek force-pushed the semantiske-farger-header branch from c73b024 to 139d605 Compare February 21, 2025 14:10
Copy link

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

@antidecaf
Copy link
Contributor

Sjekket previewen og sammenlignet med hvordan det ser ut i dag, og uten å ha gravd for mye i hvorfor ser det ut som det har skjedd noe rart med brukernavigasjonen øverst til høyre. Hovedmenyen er heller ikke sentrert lenger.

image

Dagens versjon, til sammenligning:

image

@henninghalvsb1
Copy link
Contributor

Hei, jeg skyter inn her! Vi holder på å gjøre designendringer på headeren, og det har i noen tilfeller gjort hovedmenyen off center osv., så dere kan gjerne ta inn denne først, også se om det hjelper!

#2575

@dagfrode
Copy link
Contributor

Kan du prøve å rebase denne på dev @hagenek så ser vi om centreringen og det ble løst?

Beklager sen tilbakemelding på denne.

@hagenek hagenek force-pushed the semantiske-farger-header branch from 139d605 to 24c8a7b Compare March 3, 2025 15:08
Copy link

github-actions bot commented Mar 3, 2025

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

@hagenek hagenek closed this Mar 4, 2025
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.

4 participants