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

Feil i frontend-kode i accordion - trekkspill-navigasjon på Om-skjema-sidene #210

Closed
ferkinx opened this issue Jun 12, 2024 · 5 comments
Closed
Assignees
Labels
kind/bug Something isn't working

Comments

@ferkinx
Copy link

ferkinx commented Jun 12, 2024

Description of the bug

Denne feilen gjør at flere søkemotorer - både AI-baserte og tradisjonelle - ikke finner innhold som ligger under trekkspillnavigasjonen på Om-skjema-sidene, f.eks. under "Hvem skal levere", "Mer om skjema", etc.

Feilen ble egentlig påvist av Siteimprove før vi gikk over til Monsido (som IKKE påviser feilen). Her er skjermdump av kommentaren fra Siteimprove:

image

Ber om at denne feilen prioriteres helt på topp etter prod.setting av CMS-12, siden det er veldig dumt at AI-søk ikke finner innholdet.

Kan dette være en potensiell løsning: Altinn/altinn-studio-docs#1656 ?
Eller kanskje det er flere alternativer basert på ARIA attributter som kan også løse det, og da kanskje med enda bedre kontroll?

(Denne feilen har vi diskutert på mail og på Slack i februar, og jeg trodde det lå en issue inne, men finner den ikke.)

Steps To Reproduce

Vanskelig å reprodusere, men vi har erfart det ved test av AI-søkemotoren Raffle og Altinn sin AI-assistent.

Additional Information

No response

@ferkinx ferkinx added the kind/bug Something isn't working label Jun 12, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in Team Portal Jun 12, 2024
@ferkinx ferkinx moved this from 🆕 New to 🏗 In progress in Team Portal Jun 12, 2024
@ferkinx
Copy link
Author

ferkinx commented Nov 19, 2024

@monicaolsen Kan du beskrive hva som er gjort her - og hvordan jeg ev. kan teste det?

@monicaolsen
Copy link

monicaolsen commented Nov 21, 2024

@ferkinx
Her har jeg gjort endringer i html strukturen, i hovedsak byttet ut <a> element med <button> som er semantisk riktig. Gjerne sjekk at accordions oppfører seg likt som før. Eksempel: https://prep.info.altinn.no/skjemaoversikt/bronnoysundregistrene/registrere-reelle-rettighetshavere/

Du kan også sjekke siteimprove, nå er vi nede i 18 forekomster i integration miljøet: https://my2.siteimprove.com/Accessibility/1211644/NextGen/Issue/1?conformance=0,1&siteTargetIssueKinds=1,2&wcagVersion=21&pageSegments=&ruleName=sia-r90&ruleId=90&issueKind=1&exceptTags=1,2&siteTarget.wcagVersion=21&siteTarget.conformanceLevels=0,1&siteTarget.issueKinds=1,2&siteTarget.wcagVersion=21&siteTarget.conformanceLevels=0,1&siteTarget.issueKinds=1,2

Det betyr at jeg har misset et par steder hvor accordion blir brukt men skal finne ut av det :)

@ferkinx
Copy link
Author

ferkinx commented Nov 21, 2024

@monicaolsen
Det er jo supert at en så enkel endring gjør at Siteimprove ikke regner dette som en feil lenger, men har du sett på anbefalt kode for Accordion-meny i Digdir sitt designsystem - se https://next.storybook.designsystemet.no/?path=/docs/komponenter-accordion--docs

Jeg synes det ser ut til at forslaget til frontend-kode i designsystemet ser mye enklere og mer semamtisk ut enn koden i INTE og PREP:

Image

I koden over, forstår til og med jeg hva som skjer, men ikke her:

Image

Er det noe i veien for å gjøre frontend-koden i Infoportalen mer i samsvar med anbefalingene i designsystemet?

@monicaolsen
Copy link

@ferkinx Det du ser ved å "inspisere" i nettleseren er ren HTML, mens det du ser i designsystemet er React kode. Ellers er jeg veldig enig i at det hadde vært fint å kunne bruke komponentene i designsystemet. Hadde vi brukt React kunne vi importert dem direkte. Har så smått begynt å se på å legge til React i prosjektet allerede :)

@ferkinx
Copy link
Author

ferkinx commented Nov 27, 2024

Er prod.satt i dag :)

@ferkinx ferkinx closed this as completed Nov 27, 2024
@github-project-automation github-project-automation bot moved this from Ready for production to ✅ Done in Team Portal Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

3 participants