Project

General

Profile

Bug #4116

Materialevisningsiden - ingen links i sektionen "På hjemmesiden"

Added by Gitte Barlach 7 months ago. Updated 19 days ago.

Status:
Needs code review
Priority:
Normal
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Søgning - Materialevisning

Description

Konstateret på upgrade-fbs med 4.6.0-rc6

På materialevisningssiden mangler der links til nyheden i hh. titel samt "Læs mere" knappen i sektionen "På hjemmesiden". 

Se f.eks.:

https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A52892902

Skærmbillede 2019-01-28 13.31.52.png (78.2 KB) Skærmbillede 2019-01-28 13.31.52.png der er ingen links i hverken titlen på nyheden eller knappen "Læse mere" Gitte Barlach, 01/28/2019 01:36 PM
Skærmbillede 2019-01-31 16.35.35.png (232 KB) Skærmbillede 2019-01-31 16.35.35.png Gitte Barlach, 01/31/2019 04:37 PM
Skærmbillede 2019-01-31 16.39.10.png (45 KB) Skærmbillede 2019-01-31 16.39.10.png Boksene er i samme bredde Gitte Barlach, 01/31/2019 04:48 PM
Skærmbillede 2019-01-31 16.41.03.png (70.4 KB) Skærmbillede 2019-01-31 16.41.03.png Boksene er ikke længere i samme bredde Gitte Barlach, 01/31/2019 04:48 PM
Skærmbillede 2019-01-31 16.42.09.png (65.3 KB) Skærmbillede 2019-01-31 16.42.09.png Boksene er ikke længere i samme bredde Gitte Barlach, 01/31/2019 04:48 PM

History

#1 Updated by Christel Krabbenhøft 7 months ago

  • Status changed from New to Ready for development
  • Assignee changed from Christel Krabbenhøft to Thomas Hansen
  • Target version set to Release 30 - Bugfixes

#2 Updated by Thomas Hansen 7 months ago

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

Indholdet af "På hjemmesiden" var floated left, hvilket gør at containeren ikke tæller den med i højden, men kollapser så den eferfølgdende box kommer til at overlappe indholdet, og så spiser den alle kliks.

Den rigtige løsning ville være at få ryddet op i alle de floats der er i themet, men de er brugt så mange steder at jeg ikke kunne fjerne nok til at problemet blev løst, så jeg har brugt en clearfix.

#3 Updated by Christel Krabbenhøft 7 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Thomas Hansen to Jørgen Nielsen

T-A-K, Thomas - det var dejligt hurtigt!

#4 Updated by Jørgen Nielsen 7 months ago

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

reviewet og godkendt.
@Thomas: nitpicking: husk at tilføje kommentar, hvis Scrutinizer fejl ikke er relaterede.

#5 Updated by Kasper Garnæs 7 months ago

  • Status changed from Reviewed to Technical test

Merged.

#6 Updated by Gitte Barlach 7 months ago

Testet på upgrade-fbs med 4..6.0-rc7
Der er nu igen links til relateret indhold i sektionen "På hjemmesiden" 

Hermed godkendt.

#7 Updated by Gitte Barlach 7 months ago

Okay, en lille bemærkning:

Den grå boks med med "Indhold på hjemmesiden" havde før i tiden samme bredde som de øvrige boksen/sektioner på siden. Det er den ikke længere - den er blevet smallere nu. Se vedlagte skærmdumps. 

#8 Updated by Thomas Hansen 7 months ago

Du har et skarpt øje Gitte.

(eh, hvorfor er den forsvundet fra https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A52892902 og hvorfor er der ikke noget cover på?)

Som sagt, det var et hurtigt fix fordi at gøre det ordenligt begyndte at gribe om sig, men kaster tilsyneladende andet af sig.

Der er simpelthen for mange "width: 100%; float: left" i DDBasic. De kan fikses med velplacerede clearfix'es, men moderne browsere tilbyder meget bedre layout muligheder (flexbox, CSS grid), som ikke har sjove side effekter som floating har. Når bredten er 100%, så *er* sideeffekterne den eneste effekt float har.

 

#9 Updated by Rolf Madsen 3 months ago

  • Status changed from Resolved (tag version) to Needs code review
  • Target version changed from Release 30 - Bugfixes to Release 32 - Bugfixes

Kasper skriver at https://github.com/ding2/ding2/pull/1339 ikke er merged, men stadig er relevant.

Status ændres derfor til Needs code review og Target version til Release 32 - Bugfixes.

#10 Updated by Rolf Madsen 19 days ago

Er det her ikke et duplikat af #4243 - Materialevisning - "på hjemmesiden" - link mangler til selve nyheden?

Also available in: Atom PDF