Project

General

Profile

Bug #3910

Karuseller med deaktiveret overlay viser stadig overlay + materialetype, titel og forfatter under forsidebillederne

Added by Rolf Madsen over 1 year ago. Updated about 1 year ago.

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

Description

Problemstilling

Når Ting object overlay disables under /admin/appearance/settings/ddbasic vises en div med class "no-overlay-text".

Her vises materialetype, titel og forfatter meget kluntet under forsiderne.

Formål

Vis karusellen pænt også når overlay er disabled.

Løsningsforslag

Undersøg om der er sket en fejl i seneste releases der er skyld i at stylingen er røget.

Ret fejlen, hvis den er der eller afklar med Rolf hvordan visningen skal være.

1-vanilla-indenoverlayrettes-knaperder.jpg (191 KB) 1-vanilla-indenoverlayrettes-knaperder.jpg Knap er der først Stefan Søndervang, 11/13/2018 01:30 PM
2-vanilla-overlayfra-knapvæk.jpg (185 KB) 2-vanilla-overlayfra-knapvæk.jpg Forsvinder når overlay er fra Stefan Søndervang, 11/13/2018 01:30 PM
3-vanilla-overlaytil-knapvæk.jpg (187 KB) 3-vanilla-overlaytil-knapvæk.jpg Forbliver væk efter overlay er til igen Stefan Søndervang, 11/13/2018 01:30 PM
Skærmbillede 2018-11-29 13.17.43.png (257 KB) Skærmbillede 2018-11-29 13.17.43.png Vanilla - uden gradient. Der er en Se-online knap Gitte Barlach, 11/29/2018 01:23 PM
Skærmbillede 2018-11-29 13.21.31.png (14.9 KB) Skærmbillede 2018-11-29 13.21.31.png Vanilla - uden gradient. Uheldigt med hvid farve til noget af teksten Gitte Barlach, 11/29/2018 01:24 PM
Skærmbillede 2018-12-10 18.09.40.png (260 KB) Skærmbillede 2018-12-10 18.09.40.png Karrusellen virker smallere; skulel den ikke være alignet med f.x. de andre boske på forsiden ? Gitte Barlach, 12/10/2018 06:10 PM
Skærmbillede 2018-12-10 17.32.21.png (517 KB) Skærmbillede 2018-12-10 17.32.21.png titel teksten kan lissom ikke være der Gitte Barlach, 12/10/2018 06:10 PM

Related issues

Related to DDB CMS - Bug #3944: Reserverknappen er væk når overlay er slået fra på forsidekarrusel (Test afventer #3970)Resolved (tag version)

History

#1 Updated by Rolf Madsen over 1 year ago

  • Subject changed from Karuseller med deaktiveret overlay viser materialetype, titel og forfatter to Karuseller med deaktiveret overlay viser stadig overlay + materialetype, titel og forfatter under forsidebillederne

#2 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Christel Krabbenhøft to Rasmus Høymann Laursen

Hej Rasmus. Vil I kigge på denne igen?

#3 Updated by Rasmus Høymann Laursen over 1 year ago

  • Assignee changed from Rasmus Høymann Laursen to Kristian Pedersen

#4 Updated by Kristian Pedersen over 1 year ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Kristian Pedersen to Gitte Barlach

#5 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#6 Updated by Jesper Kristensen over 1 year ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jesper Kristensen to Gitte Barlach

Reviewed og afventer release

#7 Updated by Stefan Søndervang over 1 year ago

a) når TING OBJECT OVERLAY er slået fra mangler reserver-knappen.

b) hvis TING OBJECT OVERLAY har været slået fra og man vil slå det til igen, mangler reserver-knappen stadigvæk. Den kommer tilbage efter lang tid.

Har lige testet på vanilla. Knappen var der. Slog overlay fra, så var den væk. Slog overlay til og den forblev væk i lang tid.

Jeg har vedhæftet billeder af det.

Har oprettet en ny sag: 3944

#8 Updated by Stefan Søndervang over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Technical test
  • Assignee changed from Kristian Pedersen to Stefan Søndervang

#9 Updated by Gitte Barlach over 1 year ago

  • Status changed from Technical test to Reviewed

#10 Updated by Stefan Søndervang over 1 year ago

  • Related to Bug #3944: Reserverknappen er væk når overlay er slået fra på forsidekarrusel (Test afventer #3970) added

#11 Updated by Kristian Pedersen over 1 year ago

  • Assignee changed from Stefan Søndervang to Gitte Barlach

PR indsat på det nye issue 3944

#12 Updated by Kristian Pedersen over 1 year ago

  • Status changed from Reviewed to Needs code review

#13 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#14 Updated by Gitte Barlach over 1 year ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jesper Kristensen to Gitte Barlach

#15 Updated by Kasper Garnæs over 1 year ago

  • Status changed from Reviewed to Technical test

#3944 er merged.

#16 Updated by Gitte Barlach over 1 year ago

Testet på vanilla-fbs med 4.6.0-rc2
Jeg har netop testet på vanilla-sitet da det er et jomfrueligt site hvor vi ikke har rørt ved farve-indsitllingerne. 

Jeg anser denne sag for at omhandle *styling* af materialer i karusellerne, når gradient er slået fra. 

Stylingen er meget bedre og pænt rettet op. *DOG* er det uheldigt at teksten "Af" samt udgivelsesår default er hvid. Derved kan man reelt ikke se, hvad der står. Det bør rettes og mit forslag vil være at vi følger den praksis vi lægger for dagen på selve materialevisningen. Her er farven på "Af" og udg.år med grå. 





 

#17 Updated by Kristian Pedersen about 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Kristian Pedersen to Gitte Barlach

#18 Updated by Gitte Barlach about 1 year ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#19 Updated by Jørgen Nielsen about 1 year 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 about 1 year ago

  • Status changed from Reviewed to Technical test

Merged.

#21 Updated by Gitte Barlach about 1 year ago

Testet på upgrade-fbs med 4.6.0-rc4

Farven på teksten er korrekt nu. 

a) synes stylingen ift. karusellerne er anderledes i 4.6.x
Den virker en smule smallere ned når jeg sammenligner med fx. aakb.dk - dette uanset om gradient er slået fra/til. Se vedlagte skærmdump

b) det er ikke alle titler der ser lige kønne ud når gradienten er slået fra - det er som om teksten ikke kan være der. Se vedlagte. 

c) selvom gradieren ER slået til på upgrade-fbs, så er der altså intet kryds i checkboksen under  /admin/appearance/settings/ddbasic
Der skal endvidere 2 forsøg til før jeg kan slå den til igen. Første gang vises karuseller stadig med gradient slået fra - selvom jeg har slået den fra OG clearet cashe. Men det er måske til at leve med, bare man ved det....

#22 Updated by Gitte Barlach about 1 year ago

  • Assignee changed from Gitte Barlach to Christel Krabbenhøft

#23 Updated by Christel Krabbenhøft about 1 year ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Christel Krabbenhøft to Kristian Pedersen

Hej Kristian. Jf. Gittes kommentarer efter test, vil du så kigge på denne igen? //Christel

#24 Updated by Kristian Pedersen about 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Kristian Pedersen to Gitte Barlach

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

a) Rettet

b) Med rettelsen fra punkt a, klippes tekst ikke længere, hvis den går ud over kant. Derudover er font-størrelsen er sat ned for at mindske risikoen for at ord bliver for lange.

c) Kan ikke genskabes her. Og lige for at være sikker på at der ikke er tale om en misforståelse - gradient slås fra ved at sætte kryds i tjekboksen.

#25 Updated by Gitte Barlach about 1 year ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#26 Updated by Jesper Kristensen about 1 year ago

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

Reviewed and awaiting release

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

  • Status changed from Reviewed to Technical test

Merged.

#28 Updated by Stefan Søndervang about 1 year ago

  • Assignee changed from Gitte Barlach to Stefan Søndervang

#29 Updated by Stefan Søndervang about 1 year ago

  • Status changed from Technical test to Resolved (tag version)
  • Assignee changed from Stefan Søndervang to Gitte Barlach

Testet og godkendt på vanilla. 

Jeg kan heller ikke genskabe problem c) - hverken på upgrade eller vanilla.

Also available in: Atom PDF