search

Informasjonsforvaltning / fdk-portal

Apache License 2.0
3 stars 3 forks source link

fix: skip to content #2089

Closed pooriamehregan closed 2 weeks ago

pooriamehregan commented 2 weeks ago

fixes https://github.com/Informasjonsforvaltning/fdk-portal/issues/1677

Lillebo commented 2 weeks ago

Den egentlige årsaken til problemet her er jo <base href="/" /> i <head />. Vet ikke helt hva den brukes til eller om den trengs?

pooriamehregan commented 2 weeks ago

<base href="/" /> i <head />

Jeg ser den ikke i kodebasen 🤔

Lillebo commented 2 weeks ago

<base href="/" /> i <head />

Jeg ser den ikke i kodebasen 🤔

Nei, fant den ikke jeg heller 🫤 Men jeg tror problemet blir løst hvis bare den fjernes. Hvis jeg fjerner det element fra <head> i prod (https://data.norge.no/organizations) så virker det som at skip-lenken funker som den skal. Men jeg vet ikke om det elementet brukes til noe routing-greier som gjør at andre ting da knekker hvis vi fjerner den 🤷

Men ser ut som du har rett i at det trigges en reload av tabell-dataen når skip-lenken funker som den skal. Tipper det er noe på denne siden som kanskje lytter til feil event her... 🤔

pooriamehregan commented 2 weeks ago

Oja, skjønner! Kanskje lurt å merge denne løsningen nå og så undersøke mer hvis ønskelig :) Hva synes du @Lillebo ?

Lillebo commented 2 weeks ago

Ja, enig :) Ikke noe vits å bruke masse mer tid på dette nå 👍