Project

General

Profile

Enhancement #4197

Forsidekarruseller skal kunne have en overordnet overskrift

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

Status:
Resolved (tag version)
Priority:
Normal
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Karruseller

Description

Problemstilling

  1. Karruseller har ikke en samlet overskrift. Der vises kun en "overskrift" for hvert faneblad/søgning.
  2. Description feltet benyttes ikke.

Formål

Tilføj en samlet overskrift for karrusellen der er uafhængig af faneblade/søgninger.

Løsningsforslag

  1. Tilføj feltet "Carrousel headline" (da: "Karrusel titel")  i backend, hvor karrusellen oprettes/redigeres, og vis den i frontend som en H2 overskrift.
  2. Fjern Description feltet i backend hvor karrusellen oprettes/redigeres, da feltet ikke benyttes.

Original beskrivelse

Lige nu har forsidekarruselsøgninger mulighed for at lave en overskrift, men hele forsidekarruselpanellerne har ikke en overskrift (se billede).

Det skal være muligt at lave en panel overskrift i h2. Kunne fx laves lidt som i nodelists.

Det er vigtigt i forhold til tilgængelighed at alle panellerne har overskrifter på samme niveau se bl.a.: https://platform.dandigbib.org/issues/4100.


Related issues

Related to DDB CMS - Enhancement #4100: Titlen på materialer i karrusellerne skal ikke være H2 men H3Resolved (tag version)
Related to DDB CMS - Enhancement #4171: Overskriftfeltet på alle panes skal være obligatoriskeTechnical test

History

#1 Updated by Rolf Madsen 7 months ago

  • Status changed from New to Ready for development

Det lader til at "Description" feltet ikke vises.

For hvert faneblad/søgning i karrusellen benyttes "Titel" feltet.

Kilde: https://upgrade-fbs.ddbcms.dk/

#2 Updated by Rolf Madsen 7 months ago

  • Description updated (diff)
  • Status changed from Ready for development to Needs design decision
  • Assignee changed from Christel Krabbenhøft to Stefan Søndervang
  • Target version set to Release 31 - bugfixes

@Stefan vil du godkende min beskrivelse?

#3 Updated by Rolf Madsen 7 months ago

  • Description updated (diff)

#4 Updated by Stefan Søndervang 7 months ago

  • Assignee changed from Stefan Søndervang to Rolf Madsen

Det er smukt!

#5 Updated by Christel Krabbenhøft 5 months ago

  • Assignee changed from Rolf Madsen to Preben Arentoft

#6 Updated by Thomas Hansen 4 months ago

@rolf:

Bare lige for at være sikker: I vil have panel overskrift konfigurerbar og tab titler gjort til overskrifter?

#7 Updated by Thomas Hansen 4 months ago

OK, gik op for mig at det var panel overskriften og materiale titlerne.

Førstnævte her: https://github.com/ding2/ding2/pull/1424

Sidstnævnte bliver fikset med https://platform.dandigbib.org/issues/4100, som jeg har kommenteret på.

#8 Updated by Rolf Madsen 4 months ago

  • Related to Enhancement #4100: Titlen på materialer i karrusellerne skal ikke være H2 men H3 added

#9 Updated by Rolf Madsen 4 months ago

  • Status changed from Needs design decision to Needs code review
  • Assignee changed from Preben Arentoft to Gitte Barlach

Tak for det Thomas!

#10 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#11 Updated by Jørgen Nielsen 4 months ago

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

reviewet og godkendt

#12 Updated by Kasper Garnæs 4 months ago

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

Merged.

#13 Updated by Kasper Garnæs 4 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#14 Updated by Gitte Barlach 3 months ago

Testet påvanilla-fbs med 7.x-4.8.0-rc1

Man kan ved at vælge "Tilsidesæt titel" og skrive en ny titel i feltet, der kommer frem samt vælge fontstørrelse (default er h2) få vist en overordnet overskift på ens karrusel. 

Der er altså IKKE tale om at description-feltet bliver vist, sådan som Stefan og Rolf har ønsket det, men som Thomas skriver i kommentar 7: "OK, gik op for mig at det var panel overskriften.." 

Effekten er vel den samme, tænker jeg, og måske faktisk er den løsning der er implementeret, også mere korrekt rent teknisk? 

Vil du bekræfte at vi kan godkende sagen, Stefan?

 

 

#15 Updated by Thomas Hansen 3 months ago

Oh, havde lige overset det med description. Description er leftover fra way back (tror det var en ding1 feature), ideen var vist at man kunne tilføje en lille beskrivelse der så blev vist over eller under karusellen. Men den hører specifikt til de enkelte tabs.

Jeg har vist flere gange overvejet at dræbe den når jeg har været forbi karruseller.

 

#16 Updated by Thomas Hansen 3 months ago

PR til at fjerne desciption: https://github.com/ding2/ding2/pull/1450

#17 Updated by Christel Krabbenhøft 3 months ago

  • Status changed from Technical test to Needs code review
  • Assignee changed from Stefan Søndervang to Gitte Barlach

#18 Updated by Gitte Barlach 3 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#19 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

#20 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.

#21 Updated by Stefan Søndervang 3 months ago

  • Assignee changed from Kasper Garnæs to Stefan Søndervang

#22 Updated by Stefan Søndervang 3 months ago

  • Status changed from Technical test to Ready for development
  • Assignee changed from Stefan Søndervang to Thomas Hansen

Pga 4171 er det gået op for mig, at det ikke skal være en valgmulighed om man vil override titlen. Der SKAL skrives en overskrift. Der skal ikke være en afhakningsmulighed. Feltet hvor man indtaster titlen skal bare være synlig med det samme og den skal egentlig gøre obligatorisk.

#23 Updated by Gitte Barlach 3 months ago

  • Status changed from Ready for development to Reviewed - Needs info/rework

#24 Updated by Thomas Hansen 3 months ago

@stefan

Kunne du så ikke samle dem alle sammen sammen og oprette *en* ticket på det?

Eller, Christel har alerede bedt om det for *alle* panes i #4259.

Ihvertfald er den løsningsmodel man har valgt i https://github.com/ding2/ding2/pull/1434 den forkerte, og jeg skal gladeligt uddybe hvorfor i den nye ticket hvis nogen prikker til mig når den eksisterer.

#25 Updated by Stefan Søndervang 3 months ago

  • Status changed from Reviewed - Needs info/rework to Resolved (tag version)
  • Assignee changed from Thomas Hansen to Gitte Barlach

Testet og godkendt på ding2-7.x-4.8.0-rc2.

Vi kører mine tilføjelser videre i #4171.

#26 Updated by Stefan Søndervang 3 months ago

  • Related to Enhancement #4171: Overskriftfeltet på alle panes skal være obligatoriske added

Also available in: Atom PDF