Project

General

Profile

Enhancement #4100

Titlen på materialer i karrusellerne skal ikke være H2 men H3

Added by Stefan Søndervang 7 months ago. Updated 2 months ago.

Status:
Resolved (tag version)
Priority:
Normal
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Driftsvedligehold - Teknisk tilgængelighed

Description

Denne sag er lavet på baggrund af anbefalingerne i tilgængelighedsevalueringen af DDB CMS lavet af Sensus.

Titlen på materialer i alle karusseller skal være H3 og ikke H2. Det er meget uoverskueligt for bruger der benytter skærmlæsere at alle materialerne bliver læst op på samme niveau som de andre overordnede overskrifter og ikke som indholdet i panellet.

Der skal også være en overordnet <h2> overskrift til alle karussellerne. Der er til Månedens bedst bedømte titler men ikke til den almindelig forsidekarussel.

Det burde være

<h2>I fokus
<h3>DM i Fagene
<h3>Find en læsekreds
...

<h2>Materialekarussel
<h3>Piece of cake : enkle og lækre bageopskrif...
<h3>Se mig
<h3>Sigmund
...

<h2>Nyt fra biblioteket
<h3>Kan man dø af et knust hjerte?
<h3>Nyt på hylderne
...

<h2>Det sker
<h3>Find Holger
<h3>Onsdagsværkstedet
...

Herunder er et eksempel på overskriftsstrukturen på Århus' hjemmeside.

<h2>Søgefelt
<h3>DM i Fagene
<h3>Find en læsekreds
<h3>Groovy
<h3>Auschwitz-dag
<h3>Mediernes rolle i det lokale demokrati

<h2>Piece of cake : enkle og lækre bageopskrif...
<h2>Se mig
<h2>Sigmund
<h2>Gentleman
<h2>Untitled : Louise Madsen bag facaden
<h2>Bryllup : something old, something new, so...
<h2>Grøntsagsglæde : salater & grønt tilbehør
<h2>Min ærlighed
<h2>For enden af perronen : roman
<h2>Bjørnen : roman
<h2>Faith : 1665-1918
<h2>Mødre, døtre, søstre : noveller
<h2>Lone Star
<h2>Gennem dine øjne : roman
<h2>I dine øjne : roman
<h2>Vi kunne alt : roman
<h2>Spørgsmål jeg er blevet stillet om Holocau...
<h2>Arlette : en historie vi aldrig må glemme
<h2>Advarsler før folkedrab
<h2>Dødens engel : Josef Mengeles liv og forbr...
<h2>Sorgens grundstof : roman
<h2>Jorden synger
<h2>Violinen fra Auschwitz
<h2>Auschwitz : nazisterne og den endelige løs...
<h2>Ti argumenter for at slette dine profiler ...
<h2>Mindfulness til daglig : at blive ven med ...
<h2>Frk. Skrumps nordiske middelhavskost : spi...
<h2>Jeg slukker nu! : få et sundt forhold til ...
<h2>Happy food : mad og lykke hænger sammen
<h2>Et sundt år : træning og opskrifter til he...
<h2>Dit digitale barn har brug for en voksen
<h2>Travl : lær at trives i en verden, hvor de...
<h2>Nadias dagbog
<h2>Hvem er Tristans far? : Tristan er donorba...
<h2>Barnets bedste : efter skilsmissen
<h2>Håndbog for enlige mødre : en praktisk gui...
<h2>Min far er vild med din mor
<h2>Den sammenbragte familie : vejen til succe...
<h2>Forunderlige familier
<h2>Nu er det min tur - Maries mor og mutti

<h2>Nyt fra biblioteket
<h3>Kan man dø af et knust hjerte?
<h3>Nyt på hylderne
<h3>Aarhus Bibliotekernes forårsprogram er på gaden
<h3>Ny dansk litteratur
<h3>Markante forfattere - på engelsk

<h2>Det sker
<h3>Find Holger
<h3>Onsdagsværkstedet
<h3>Studiecafé på Dokk1


Related issues

Related to DDB CMS - Enhancement #4099: Tabrolls skal have en H2 over sigResolved (tag version)
Related to DDB CMS - Enhancement #4197: Forsidekarruseller skal kunne have en overordnet overskriftResolved (tag version)

History

#1 Updated by Stefan Søndervang 7 months ago

#2 Updated by Christel Krabbenhøft 7 months ago

  • Assignee changed from Christel Krabbenhøft to Benjamin Rasmussen

#3 Updated by Benjamin Rasmussen 7 months ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Benjamin Rasmussen to Stefan Søndervang

Efter en helt masse gravearbejde fandt jeg endelig ud af hvor det bliver sat:

https://github.com/ding2/ding2/blob/master/modules/ting/ting.field.inc#L410

 

Problemet er bare at hvis jeg retter det her, så vil det slå igennem på hele sitet, inklusiv fx. søgesiden, og det er jo ikke så fedt.
Det er også muligt at der er en masse ting der kan gå i stykker hvis jeg gør det pga. stylingen.

 

Jeg tænkte dog på - kunne et alternativ ikke være at ændre titlen på karouselerne fra H2 til H1?

#4 Updated by Stefan Søndervang 7 months ago

  • Status changed from Need more info to Needs design decision
  • Assignee changed from Stefan Søndervang to Benjamin Rasmussen

Nej. Det må kun være en H1, som er hovedoverskriften pga SEO. 

Er der en anden måde at snyde den? 

#5 Updated by Benjamin Rasmussen 7 months ago

  • Status changed from Needs design decision to Need more info
  • Assignee changed from Benjamin Rasmussen to Stefan Søndervang

Hej Stefan

Er du sikker på at det er rigtigt? :)

Det har været standard længe at man kun skal have et enkelt tag, men med HTML5 er det vist ret normalt at have flere H1 tags på sider - det er både semantisk og SEO korrekt ud fra hvad jeg kan forstå

Se fx:

http://www.suvaance.com/microblog/how-google-considers-multiple-h1-tags-in-one-page/

 

og lidt længere kilde:

https://webdesign.tutsplus.com/articles/the-truth-about-multiple-h1-tags-in-the-html5-era--webdesign-16824

#6 Updated by Stefan Søndervang 7 months ago

  • Assignee changed from Stefan Søndervang to Benjamin Rasmussen

De kan da ikke bare ændre det! Det bryder den gamle mand i mig ikke om!

Uanset hvad er det et problem. Hvis vi skal ændre den til H1 skal alle de andre H2'ere ændres til H1 fx Nyt fra biblioteket, Det sker etc. Og så skal alle H3'erne ændres til H2. 

Ellers virker det som om at materialekarrusellerne er overoverskriften for alt indholdet efterfølgende plus materialerne er på niveau med nyhedspanelet og ikke nyhederne. Panels skal være samme H, og indhold skal være samme H. Ellers har vi lavet samme problem på en ny måde.

Eksempel (dem i bold burde være samme niveau):

<h2>I fokus

<h3>DM i Fagene
<h3>Find en læsekreds
 

<h1>Materialekarussel

<h2>Piece of cake : enkle og lækre bageopskrif...
<h2>Se mig
<h2>Sigmund
 

<h2>Nyt fra biblioteket

<h3>Kan man dø af et knust hjerte?
<h3>Nyt på hylderne
 

<h2>Det sker

<h3>Find Holger
<h3>Onsdagsværkstedet

#7 Updated by Benjamin Rasmussen 7 months ago

God pointe Stefan :)

Ok, jeg prøver lige at grave lidt, og se om jeg kan lave en eller anden undtagelse for at fikse det her, men jeg synes at vi skal gå varskoet med undtagelser da det er præcis hvad der bider os bag i senere 

#8 Updated by Benjamin Rasmussen 7 months ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Benjamin Rasmussen to Gitte Barlach

PR er klar:

https://github.com/ding2/ding2/pull/1361

 

Jeg er egentlig ikke meget for løsningen, fordi det føles som endnu mere kompleksitet til et site som allerede er meget kompleks - men det er den eneste måde det kan løses på

#9 Updated by Gitte Barlach 6 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#10 Updated by Jørgen Nielsen 6 months ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

#11 Updated by Thomas Hansen 4 months ago

Karuseller renderer materialer i display_mode teaser, som ikke bliver brugt til andet. Så i stedet for det her hack er det smartere at udvide ting_title_default field formatteren så man kan angive om det skal være h2 eller h3 (evt. flere og ingen), og så konfigurere det for teaser display moden.

 

#12 Updated by Rolf Madsen 4 months ago

  • Related to Enhancement #4197: Forsidekarruseller skal kunne have en overordnet overskrift added

#13 Updated by Rolf Madsen 4 months ago

  • Status changed from Reviewed to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Christel Krabbenhøft

@Christel, qua Thomas' kommentar skal vi lige have den her forbi Benjamin igen.

#14 Updated by Christel Krabbenhøft 4 months ago

  • Assignee changed from Christel Krabbenhøft to Benjamin Rasmussen

#15 Updated by Benjamin Rasmussen 4 months ago

  • Assignee changed from Benjamin Rasmussen to Thomas Hansen

#16 Updated by Thomas Hansen 4 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review

#17 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Thomas Hansen to Jørgen Nielsen

#18 Updated by Jørgen Nielsen 3 months ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

#19 Updated by Kasper Garnæs 3 months ago

  • Status changed from Reviewed to Technical test
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

#20 Updated by Kasper Garnæs 3 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#21 Updated by Stefan Søndervang 2 months ago

  • Status changed from Technical test to Resolved (tag version)

Testet og godkendt på fbs-upgrade på ding2-7.x-4.8.0-rc1.

Det ser bare godt ud!

Also available in: Atom PDF