Project

General

Profile

Bug #3084

Lister med tags - rod i visning af rækkefølgen samt ikke klikbare links

Added by Kasper Hagel Madsen almost 2 years ago. Updated about 1 year ago.

Status:
Resolved (tag version)
Priority:
Normal
Assignee:
-
Estimated time:
URL med eksempel:
https://frederiksberg-stg.ddbcms.dk/tags/krimier-i-raekkefoelge
Kategorier:
Inspiration - Nyheder

Description

Vi er inde på vores staging side, her inden lanceringen af det nye design og opdager, at nogle fejl på denne side med en liste af nyheder med samme tags:

https://frederiksberg-stg.ddbcms.dk/tags/krimier-i-raekkefoelge

Fejl 1. Rækkefølgen på siderne: På produktionssiden er der en rækkefølge https://fkb.dk/tags/krimier-i-raekkefoelge

men en anden på staging siden hvor nyheden 'Krimier i rækkefølge S-Å' ikke ligger sidst som den skal.

 

Fejl 2. Links er ikke klikbare: Der er ikke link fra titlen og til siden, hvilket der er på produktionssiden. Det ser ud til at der er link fra billedet, men der skal vel også være fra titlen!?


Related issues

Has duplicate DDB CMS - Bug #3258: Tagsamlesider - Bedre visning i rækkefølgen af nyheder, arrangementer og sider Closed

History

#1 Updated by Christel Krabbenhøft almost 2 years ago

  • Status changed from New to Ready for development
  • Assignee set to Rasmus Høymann Laursen
  • Target version set to Release 27 - Bugfixes (B14)

#2 Updated by Rasmus Høymann Laursen almost 2 years ago

  • Assignee changed from Rasmus Høymann Laursen to Steen Holten-Andersen

#3 Updated by Kasper Hagel Madsen almost 2 years ago

Desuden er titlen på siden https://frederiksberg-stg.ddbcms.dk/tags/krimier-i-raekkefoelge 

blevet til 'Side i kategorien Krimier i rækkefølge'

På produktionssiden https://fkb.dk/krimier-i-raekkefoelge hedder den blot 'Krimier i rækkefølge', hvilket er en bedre ordlyd.

#4 Updated by Steen Holten-Andersen almost 2 years ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

PR: https://github.com/ding2/ding2/pull/903

Har defineret og stylet ding_page teaser, og ding_page teaser_no_overlay viewmodes. 

Har tilføjet 'Remove overlay' setting på ding_page teasers.

Har fjernet Masonry layout på tag-landings, og de enkelte nodes i hver gruppe sorteres efter Publishing Date.

Et tidligere commit har ændret på titlen (og er tidligere godkendt) - titlen vises som overskrift for hver gruppe, defineret af indholdstype:

ding_page: Page in the category [TAGNAME]

ding_news: News in the category [TAGNAME]

ding_event: Event in the category [TAGNAME]

Hvis andet ønskes, skal der tages en beslutning om hvordan det løses i stedet.

 

#5 Updated by Jørgen Nielsen almost 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

Der er lidt whitespace nitpicking. Ellers OK

#6 Updated by Steen Holten-Andersen almost 2 years ago

  • Assignee changed from Steen Holten-Andersen to Jørgen Nielsen

@Jørgen - Hvor finder du det ekstra whitespace? Syntes umiddelbart ikke jeg finde noget her: https://github.com/ding2/ding2/pull/903/files

#7 Updated by Jørgen Nielsen almost 2 years ago

  • Assignee changed from Jørgen Nielsen to Steen Holten-Andersen

Jeg har tilføjet et par kommentarer i files

#8 Updated by Steen Holten-Andersen almost 2 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Steen Holten-Andersen to Jørgen Nielsen

#9 Updated by Jørgen Nielsen almost 2 years ago

  • Assignee changed from Jørgen Nielsen to Steen Holten-Andersen

Der var vist en søg/erstat, der gik i kage ;-)

#10 Updated by Steen Holten-Andersen almost 2 years ago

  • Assignee changed from Steen Holten-Andersen to Jørgen Nielsen

New PR: https://github.com/ding2/ding2/pull/903

@Søren - ja tror der måske havde sneget sig en lille Nisse ind i min text-editor. Jeg har rebootet, og installeret et anti-nisse program. Beklager.

#11 Updated by Jørgen Nielsen almost 2 years ago

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

reviewet og godkendt

#12 Updated by Jesper Kristensen almost 2 years ago

  • Status changed from Reviewed to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

Det er desværre nogle issue med kode stillen der magler at blive rette endnu.

#13 Updated by Steen Holten-Andersen almost 2 years ago

Jesper Kristensen skrev:

Det er desværre nogle issue med kode stillen der magler at blive rette endnu.

Hej Jesper - har skrevet en kommentar på Github. Vil du lige tage stilling til den - så kan jeg lukke det her issue?

#14 Updated by Steen Holten-Andersen almost 2 years ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Steen Holten-Andersen to Jesper Kristensen

@Jesper - skrev dette spm. på Github:

The markup in node--ding-page--view-mode-teaser.tpl.php (and --view-mode-teaser-no-overlay) is based on the markup already in use in node--ding-group--view-mode-teaser.tpl.php (and node--ding-group--view-mode-teaser-no-overlay.tpl.php)

If we change the preprocess/markup style in the page-teasers, maybe it should be changed also in group-teaser - to keep things clean?

Syntes du vi skal ændre kode-stilen mht. baggrundsbilleder - på både node--ding-page og node--ding-group?

#15 Updated by Rolf Madsen almost 2 years ago

  • Status changed from Need more info to Needs code review

Jeg ændrere status fra Needs more info til Needs code review for at vi ikke glemmer hvor langt den er i forløbet.

#16 Updated by Steen Holten-Andersen almost 2 years ago

  • Assignee changed from Jesper Kristensen to Gitte Barlach

Hej Gitte - tildelte den her til Jesper, da jeg havde nogle spm. til hans kommentarer - men der har den ligget et godt stykke tid, og nu er der dukket et andet issue op som indeholder det samme: https://platform.dandigbib.org/issues/3258, vil du se på at få sendt den til den rette så vi kan komme i hus med dette issue?

#17 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#18 Updated by Rolf Madsen almost 2 years ago

  • Target version changed from Release 27 - Bugfixes (B14) to Release 29-2 - Bugfixes (7.x-4.5.0)

#19 Updated by Rolf Madsen over 1 year ago

  • Has duplicate Bug #3258: Tagsamlesider - Bedre visning i rækkefølgen af nyheder, arrangementer og sider added

#20 Updated by Jesper Kristensen over 1 year ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jesper Kristensen to Steen Holten-Andersen

Jeg har skrevet kommentar til dette PR angående at den ikke overholder kode standarden i template filerne specialt.

Være venlige at gennemgå disse og rette dem. Derefter skal der lige en rebase til for at match nyest core.

#21 Updated by Steen Holten-Andersen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Steen Holten-Andersen to Jesper Kristensen

Rebased og code-style rettet.

Har også rettet i code-style tilhørende ddbasic_preprocess__node__ding_group og hertil node--ding-group--view-mode--teaser.tpl.php - så de matcher stilen fra ddbasic_preprocess__node__ding_page og hertil node--ding-page--view-mode--teaser.tpl.php

#22 Updated by Jørgen Nielsen over 1 year ago

  • Assignee changed from Jesper Kristensen to Jørgen Nielsen

#23 Updated by Jørgen Nielsen over 1 year ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Steen Holten-Andersen

Der er stadigvæk et par Scrutinizer fejl - kan du rette dem?

#24 Updated by Steen Holten-Andersen over 1 year ago

  • Assignee changed from Steen Holten-Andersen to Jørgen Nielsen

Jeg har lidt svært ved at finde ud af hvad fejlene er:

1. I views-view-unformatted--tags-list.tpl.php - er det eneste jeg har fjernet de klasser fra <div class="view-elements"> som introducerede masonry griddet.

2. I node--ding-page--view-mode--teaser-no-overlay.tpl.php - syntes jeg efter min bedste overbevisning at tingene ser korrekt ud.

 

Kan du lede mig lidt nærmere?

Mvh

#25 Updated by Jørgen Nielsen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach


Hmm... såvidt jeg kan google mig til, er det en kendt fejl. Valideringen fejler, hvis man bruger "alternate syntax" inde i html.
Jeg tror, vi trygt kan ignorere det (elles: lad være med at bruge "alternate syntax" ;-)

Reviewed og godkendt

 

#26 Updated by Kasper Garnæs about 1 year ago

  • Status changed from Reviewed to Technical test

PR'et mergede ikke automatisk. Jeg har forsøgt at merge det i hånden.

#27 Updated by Rolf Madsen about 1 year ago

  • Assignee changed from Gitte Barlach to Rolf Madsen

#28 Updated by Rolf Madsen about 1 year ago

  • Status changed from Technical test to Resolved (tag version)
  • Assignee deleted (Rolf Madsen)

Testet og godkendt.

Udgangspunkt for test: https://upgrade-29.ddbcms.dk/tags/test

Sortérinen af noderne efter titel a-z, mener jeg passer til den specifikke usecase Frederiksberg Bibliotekerne har på https://frederiksberg-stg.ddbcms.dk/tags/krimier-i-raekkefoelge, men i de fleste andre tilfælde vil en sortering efter nyeste først være  den mest relevante.

Resultater

  1. Noderne er sorteret efter nyeste øverst.
  2. Links er klikbare og leder til de pågældende sider
  3. Sidetitlen er sammensat i forhold til gældende design "ding_page: Page|News|Event in the category [TAGNAME]", som der er refereret til i https://platform.dandigbib.org/issues/3084#note-4.

Also available in: Atom PDF