Project

General

Profile

Bug #4640

Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra (overlap til #4562; gen-testes når denne er løst)

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

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

Description

Konstateret på upgrade-fbs med rel. 5.0.1

Når overlay er slået fra, går visningen i forsidekarrusellen helt galt for materialer, der enten har en lang titel eller en lang indholdsnote (eller begge dele). Reservér-knap og tekst står oveni hinanden, og teksten fylder alt for meget. 
Se vedhæftede eksempler - karrusellen "Biblioteket anbefaler materialer til dig" 

Der ønskes i stedet en visning hvor hhv. titel og indholdsnote trunkeres efter x-antal tegn.

Skærmbillede 2019-11-26 kl. 16.12.06.png (254 KB) Skærmbillede 2019-11-26 kl. 16.12.06.png Gitte Barlach, 11/26/2019 04:57 PM
Skærmbillede 2019-11-26 kl. 16.12.24.png (403 KB) Skærmbillede 2019-11-26 kl. 16.12.24.png Gitte Barlach, 11/26/2019 04:57 PM
Skærmbillede 2020-05-14 kl. 17.00.19.png (156 KB) Skærmbillede 2020-05-14 kl. 17.00.19.png indholdsnoten Gitte Barlach, 05/14/2020 05:19 PM
Skærmbillede 2020-05-14 kl. 16.56.12.png (259 KB) Skærmbillede 2020-05-14 kl. 16.56.12.png indholdsnoten Gitte Barlach, 05/14/2020 05:20 PM
Skærmbillede 2020-05-14 kl. 16.51.45.png (510 KB) Skærmbillede 2020-05-14 kl. 16.51.45.png reserver-knap mangler Gitte Barlach, 05/14/2020 05:20 PM
Skærmbillede 2020-05-14 kl. 16.53.30.png (804 KB) Skærmbillede 2020-05-14 kl. 16.53.30.png grå baggrund - skal ikke være der Gitte Barlach, 05/14/2020 05:21 PM
Skærmbillede 2020-05-14 kl. 17.08.13.png (255 KB) Skærmbillede 2020-05-14 kl. 17.08.13.png paragraphs af typen "Enkelt materiale, ½ spalte Gitte Barlach, 05/14/2020 05:22 PM
Skærmbillede 2020-05-14 kl. 17.09.12.png (161 KB) Skærmbillede 2020-05-14 kl. 17.09.12.png paragraphs af typen "Enkelt materiale, ½ spalte Gitte Barlach, 05/14/2020 05:22 PM

Related issues

Related to DDB CMS - Bug #4562: Performance på sektioner er meget langsom TESTES MED #4622Resolved (tag version)

History

#1 Updated by Gitte Barlach 7 months ago

  • Subject changed from uheldig visning af visse materialer når overlay er slået fra to Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra

#2 Updated by Gitte Barlach 7 months ago

  • Assignee set to Christel Krabbenhøft

#3 Updated by René Krøll 5 months ago

Er der nogen tidsplan på den her?

Vi har slået overlay til, og vi vil gøre som Randers med at have en karrusel i toppen af siden med bøger, men det er ikke ligefrem det mest indbydende at vise brugerne.

 

#4 Updated by Rolf Madsen 5 months ago

  • Status changed from New to Ready for development
  • Target version set to Release 31-2 - Bug fixes (7.x-5.?.?)

Vi må lige se hvor vi kan få den ind.

#5 Updated by Christel Krabbenhøft 4 months ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Christel Krabbenhøft to Gitte Barlach

#6 Updated by Benjamin Rasmussen about 2 months ago

  • Status changed from Need more info to Development
  • Assignee changed from Gitte Barlach to Benjamin Rasmussen

Antager det var en fejl at den var sat til "Needs more info", men jeg går altså i gang med den nu :)

#7 Updated by René Krøll about 2 months ago

Så smukt Benjamin :)

#8 Updated by Gitte Barlach about 2 months ago

Hej Benjamin
Når du nu er igang med at kigge på materialevisning med og uden overlay, vil jeg lige gøre dig opmærksom på de fejl vi oplever i 7.x-6.1.0-beta2 med karruselvisning; de er beskrevet i note 21, 22 24 i issue #4562

#9 Updated by Benjamin Rasmussen about 2 months ago


Hej Gitte

Jeg har kigget lidt i https://platform.dandigbib.org/issues/4562 men jeg tror altså ikke jeg er den rigtige til at kigge på det.
Jeg kan godt se at det ser galt ud, men jeg har ingen ide om hvad det kræver at fikse

Det er ihvertfald en del mere kompliceret end jeg har tid sat af til at kigge på lige nu - beklager :(

#10 Updated by Gitte Barlach about 2 months ago

Hej Benjamin
Det er helt ok; #4562 er jo også tildelt til Kasper; jeg ville bare sikre mig at I ikke begge sidder og arbejder på noget der måske ville vise at at  den samme fejl når det kommer til stykket. 

#11 Updated by Benjamin Rasmussen about 2 months ago

  • Status changed from Development to Needs code review
  • Assignee changed from Benjamin Rasmussen to Gitte Barlach

Hej Gitte

Ah, ja i så fald så er det dejligt du lige nævnte det :)

Jeg har et PR klart:

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

Ifbm. PR'et er der jo oprettet et test site hvor du kan prøve det af allerede nu:

https://4640-tabroll-overlay-oymeh7i-ton6gvqucinhw.eu-4.platformsh.site/

 

Jeg har også fikset en lille smule omkring knapperne, men jeg har altså ingen ide om hvad der sker for den der manglende beskrivelsestekst

Her er billeder fra mit lokalmiljø, før og efter mit fix:

FØR

 

EFTER

 

#12 Updated by Gitte Barlach about 2 months ago

Pre-testet på platform.sh
Jeg tror vi skal have kigget på denne igen; dog er der jo overlap til #4640

a) med gradient slået TIL:
a1) indholdsnoten. Jeg kan desværre slet ikke se indholdnoten; men det er jo heller ikke fixet endnu. 
a2) pile-knapperne. Rettet nu. Godkendt. 
NY fejl : resérver-knap mangler. Jeg har kun en "Læs mere" knap; jeg har tjekket konfigs og der er reserverknap på f.eks. materialevisning. Sp mener ikke at den manglende knap skyldes konfigs. 


b) med gradient slået FRA:
b.1) over og under forsidebilledet er der så vidt jeg kan se stadig en grå baggrund. Den skal ikke være der.
b.2) noten bliver desværre stadig ikke vist på den rigtige måde. Se vedlagte skærmbillede. Men er jo heller ikke fixet endnu.
b.3) overlay, der indeholder knapperne "Læs mere" og "Reservér" passer ikke rigtigt i størrelsen. Ser umiddelbart ud til at være løst.  


paragraphs af typen "Enkelt materiale, ½ spalte" - 
@tue: se vedlagte skærmdumps - kan du godkende den visning?

@Benjamin: så vidt jeg kan se af dine eksempler har du også visning af knappe "Andre materialetyper" med. Den skal dog ikke vises her på dette niveau (og jeg får den altså heller ikke at se, ligesom jeg ikke ser reserver-knappen)

#14 Updated by Gitte Barlach about 2 months ago

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

#15 Updated by Christel Krabbenhøft about 2 months ago

  • Assignee changed from Christel Krabbenhøft to Kasper Garnæs

#16 Updated by Tue Gaston about 2 months ago

Gitte Barlach wrote:

paragraphs af typen "Enkelt materiale, ½ spalte" - 
@tue: se vedlagte skærmdumps - kan du godkende den visning?

De er godkendt.
De andre er ikke så kønne, men "Enkelt materiale, ½ spalte" er jo nydelig.

#17 Updated by Benjamin Rasmussen about 2 months ago

@Gitte:

 

a1, b1, b2:
Det med den grå baggrund og manglende noter hænger sammen.
Det er ikke forudsaget at de ændringer jeg har lavet i mit pr, og jeg valgte med vilje ikke at kigge på det, da det var den del jeg ikke kendte nok til fra den andet issue :)

 

a2, b3:
Jeg antager at jeg skal læse det som at det er godkendt af dig (?)

 

"resérver-knap mangler.":

Jeg har kun ændret på styling, så hvis reserver knappen mangler antager jeg at det er et problem med platform.sh miljøet.
Er der et konkret sted/url hvor du ved at reserver burde være der, men ikke er der?

Husk dog at denne opgave egentlig kun handlede om at teksten kunne lægge sig under knapperne :)

 

"så vidt jeg kan se af dine eksempler har du også visning af knappe "Andre materialetyper" med. Den skal dog ikke vises her på dette niveau "

På mit lokalmiljø kunne jeg få den knap frem. Lad vær med at læse for meget ind i det - jeg tror at det er fordi mit lokalmiljø er lidt underligt.
Det var bare for at vise at selv hvis der er 2 niveauer af knapper, så vil de stadig kunne vises uden at det hele knækker :)

 

#18 Updated by Gitte Barlach about 1 month ago

  • Subject changed from Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra to Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra (overlap til #4640)

#19 Updated by Tue Gaston about 1 month ago

  • Related to Bug #4562: Performance på sektioner er meget langsom TESTES MED #4622 added

#20 Updated by Gitte Barlach about 1 month ago

  • Subject changed from Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra (overlap til #4640) to Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra (overlap til #4562; gen-testes når denne er løst)
  • Status changed from Reviewed - Needs info/rework to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

#21 Updated by Gitte Barlach 15 days ago

Jeg tester denne.

#22 Updated by Gitte Barlach 15 days ago

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

Testet og godkendt - se mine test kommentarer #4562. Bemærk dog at selve koderettelsen iflg. Simon ligger i #4622

Also available in: Atom PDF