Project

General

Profile

Bug #4415

"Se online" knap er grå. Den burde være sort.

Added by Tue Gaston 8 months ago. Updated 6 months ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Nyheder, Inspiration - Arrangementer, Inspiration - Sektioner og temaer

Description

Efter opgradering til rel. 7.x-4.8.0-rc2 har "Se online" knapperne skiftet farve:

Fejlen er konstateret på https://stg.aakb.dk/nyheder/inspiration/engelske-eboeger-biografier-med-mere (se rigtig visning her: https://www.aakb.dk/nyheder/inspiration/engelske-eboeger-biografier-med-mere ) og på upgrade: https://upgrade-fbs.ddbcms.dk/nyheder/boeger/sportsstjerner-bag-facaden

History

#1 Updated by Christel Krabbenhøft 7 months ago

  • Assignee changed from Gitte Barlach to Rasmus Høymann Laursen

Hej Rasmus. Vil I fikse denne? //Christel

#2 Updated by Rasmus Høymann Laursen 7 months ago

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

#3 Updated by Kristian Pedersen 7 months ago

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

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

Farven på knappen er ændret til sort

#4 Updated by Gitte Barlach 7 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#5 Updated by Kasper Garnæs 7 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework

Kristian: Jeg er usikker på løsningen ift. problemet.

Titlen på dene ticket er "Se online" knap er grå. Den burde være sort. I den foreslåede ændring ændres farven på se-online knappen (med klassen button-see-online) og reserveringsknappen (med klassen reserve-button)

Er det noget der efterfølgende er blevet aftalt eller hvordan?

 

#6 Updated by Kasper Garnæs 7 months ago

  • Assignee changed from Kasper Garnæs to Kristian Pedersen

#7 Updated by Christel Krabbenhøft 7 months ago

Det er kun farven på "Se online"-knappen, der er forkert.

#8 Updated by Kristian Pedersen 7 months ago

Det er rigtigt, at issuet kun beskrev et problem med grå "Se online" knap.

Men efter, hvad jeg kunne finde frem til er problemet opstået ved, at baggrundsfarven er blevet ændret til grå på knapper med både klassen "button-see-online" og "reserve-button". Begge disse var førhen sorte. Jeg kunne ikke finde frem til, hvornår dette er sket.

I materialevisningen på skærmbilledet i dette issue (Materialeliste på nyhed), overskrives baggrundsfarven på "reserve-button" til fra sort til grå - det gør den stadig.

Men i basic stylingen på "reserve-button" bør baggrundsfarven være sort - ellers bliver reserver-knapper i materialers teaservisning grå, hvilket den ikke bør være (se vedhæftede billede).

Jeg håber at det giver mening. Og jeg kan godt se, at der manglede en forklaring i commit'et

#9 Updated by Tue Gaston 7 months ago

Kristian Pedersen wrote:

Det er rigtigt, at issuet kun beskrev et problem med grå "Se online" knap.

Men efter, hvad jeg kunne finde frem til er problemet opstået ved, at baggrundsfarven er blevet ændret til grå på knapper med både klassen "button-see-online" og "reserve-button". Begge disse var førhen sorte. Jeg kunne ikke finde frem til, hvornår dette er sket.

I materialevisningen på skærmbilledet i dette issue (Materialeliste på nyhed), overskrives baggrundsfarven på "reserve-button" til fra sort til grå - det gør den stadig.

Men i basic stylingen på "reserve-button" bør baggrundsfarven være sort - ellers bliver reserver-knapper i materialers teaservisning grå, hvilket den ikke bør være (se vedhæftede billede).

Jeg håber at det giver mening. Og jeg kan godt se, at der manglede en forklaring i commit'et


Kristian har ret, og alle vi andre tager fejl, og jeg var heller ikke vaks nok ved havelågen, da jeg oprindeligt oprettede issue't.
Det er naturligvis ikke kun "Se online" knappen der skal være sort - det er også "Reservér" knappen.

#10 Updated by Gitte Barlach 7 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Christel Krabbenhøft to Kasper Garnæs

#11 Updated by Kasper Garnæs 7 months ago

  • Status changed from Needs code review to Reviewed

Tak for opklaring alle.

Reviewed og godkendt.

#12 Updated by Kasper Garnæs 7 months ago

  • Status changed from Reviewed to Technical test

#13 Updated by Kasper Garnæs 6 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#14 Updated by Gitte Barlach 6 months ago

testet på upgrade-fbs med Release - 7.x-4.8.0-rc5

a) farven på reservérknappen er nu sort (gælder værkvisning og materialevisning). Godkendt. 
b) farven på se-online knappen er også sort. Godkendt.
c) der mangler reserver-knapper på materialer, der er vedhæftet en nyhed, uanset om der er valgt visning som materialeliste eller ej. Ikke godkendt. (se vedhæftede skærmdumps)

eksempel: https://upgrade-fbs.ddbcms.dk/nyheder/anbefalinger/10-nordiske-krimier-som-du-maske-ikke-har-opdaget-endnu-0

#15 Updated by Christel Krabbenhøft 6 months ago

  • Assignee changed from Gitte Barlach to Kristian Pedersen

Hej Kristian. Jf. Gittes resultat fra testen vil du så kigge på denne igen? //Christel

#16 Updated by Gitte Barlach 6 months ago

Hej Kristian 

Har du mulighed for at prioritere denne? vi kan desværre ikke release før der igen er reserver knapper på tilknyttede materialer til en nyhed. 

#17 Updated by Philip Birk-Jensen 6 months ago

  • Assignee changed from Kristian Pedersen to Gitte Barlach

Den manglende reserver knap lader til ikke at være relateret dette issue.

Det ser ud til fejlen er kommet med i #1460, nærmere dette commit: https://github.com/ding2/ding2/commit/3314e63e29d8026825392dc9bf83db33e146d071

Problemet er /js/ding_reservation_reservability.js tilføjes via #attached, så når referencer loades igennem ting_reference/ajax/%/% bliver denne js fil ikke tilføjet.

Uden nogen indsigt i issue 1460 ville et quickfix være altid tilføje /js/ding_reservation_reservability.js, eller skrive ting_reference/ajax/%/% om så den bruger Drupals AJAX API.
Men jeg kender som sagt ikke til 1460, så der er sikkert andre der har noget input her.

#18 Updated by Simon Holt 6 months ago

Jeg foreslår vi flytter problemet med manglende reserveringsknap til #1460? Er alligevel ved at kigge på nogle kommentarer efter code review, så det vil give god mening at kigge på det i samme omgang.

#19 Updated by Gitte Barlach 6 months ago

  • Status changed from Reviewed - Needs info/rework to Resolved (tag version)

Tak for tilbagemelding, Phillip og Simon. Jeg godkender denne; manglende res.knap håndteres i #1460, som foreslået at Simon.

#20 Updated by Simon Holt 6 months ago

Efter test ifb med #1460 ser det ud til at reserveringsknappen nu er grå. Se vedhæfning.

Så vidt jeg kan læse mig til burde den være sort?

#21 Updated by Christel Krabbenhøft 6 months ago

Hej Simon. Jo, den burde være sort :-)

#22 Updated by Simon Holt 6 months ago

Hej Christel. Alarmen er afblæst. Havde glemt at køre en "drush cgen" efter jeg havde trukket det seneste kode ned. My bad. Alt er i orden og knappen er sort :)

#23 Updated by Christel Krabbenhøft 6 months ago

Fedt, Simon! Tusind tak!

Also available in: Atom PDF