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

More correct heading levels on topic pages #1293

Closed
wants to merge 1 commit into from

Conversation

Jonas-C
Copy link
Contributor

@Jonas-C Jonas-C commented Aug 29, 2022

Relatert til, men fikser ikke, https://github.com/NDLANO/Issues/issues/3225

Jeg er usikker på om dette er "riktig" med tanke på emner og underemner. Skal et underemne være en h3, fordi emnet det tilhører er en h2, eller skal begge være h2? Dette er uansett mer riktig enn nåværende løsning, der alt er h1.

@Jonas-C Jonas-C requested a review from a team August 29, 2022 11:58
Copy link
Member

@gunnarvelle gunnarvelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vil dette sette h3 fra nivå tre og nedover?

@Jonas-C
Copy link
Contributor Author

Jonas-C commented Aug 29, 2022

Vil dette sette h3 fra nivå tre og nedover?

Nei. Strukturen blir som følger
h1 -> Fag
h2 -> Emne
h3 -> Underemner (over NavigationBox)
h2 -> Annet Emne
h3 -> Underemner (for Annet Emne, over NavigationBox)

Vet ikke om dette nødvendigvis blir bedre enn nåværende løsning. Hver topic-seksjon er forsåvidt wrappet i en <section />, så kanskje det er greit med h1?

Tanken min var at emner ligger under et fag, og derfor bør være en mindre betydelig heading. Hvorvidt dette gir mening i kontekst av siden er jeg usikker på, i og med at det er vanskelig å skille mellom det faktum at man er på et nøstet nivå i en side som egentlig omhandler et fag. Dette er strengt tatt semantikk, og er muligens et valg som NDLA må få et innspill i.

@gunnarvelle
Copy link
Member

Dette må vi sjekke nærmere før vi går videre med. Mulig det korrekte er å ha h1 i kvar section.

@Jonas-C Jonas-C closed this May 30, 2024
@Jonas-C Jonas-C deleted the improve-heading-structure-in-topic-pages branch May 30, 2024 08:32
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.

None yet

2 participants