Project

General

Profile

Bug #1229

Reserverknap for materiale, der er hjemme, er brun

Added by Majken Fransø about 5 years ago. Updated about 3 years ago.

Status:
Resolved (tag version)
Priority:
Normal
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Søgning - Materialevisning

Description

I Vallensbæk søgebase er reservérknappen for materiale, der er hjemme brun. De fleste andre steder er den grøn. Hvilket den jo også bør være. Er det noget vi selv kan styre? Jeg kan ikke se hvor.
Jeg har lavet en lille undersøgelse og set, at i Ishøj er knappen også brun og i Morsø og Haderslev er den nærmest mørkerød! sidstnævnte er da helt ulogisk

Reservér knap brun.jpg (144 KB) Reservér knap brun.jpg Majken Fransø, 05/18/2015 09:09 AM
Reserver knappen.jpg (1.64 KB) Reserver knappen.jpg Betina Nakel Thorsager, 06/03/2015 11:33 AM
reserverknap.jpg (16.1 KB) reserverknap.jpg Svend Lentz Jørgensen, 06/03/2015 05:22 PM

History

#1 Updated by Rolf Madsen about 5 years ago

  • Status changed from New to Open (waiting)
  • Target version set to DDB CMS 2015 3. opgradering

Jeg formoder dette er en mindre opgave og formentlig et spørgsmål om kofiguration, men jeg har ikke været i stand til at finde en løsning på det og er nødt til at have en udvikler til at analysere det.

#2 Updated by Betina Nakel Thorsager almost 5 years ago

I Allerød er vores reservérknap en mystisk kombination af blå og rød - men kun for titler, der er udlånte. Dem der er hjemme, er fint røde...

#3 Updated by Svend Lentz Jørgensen almost 5 years ago

Morsø Folkebibliotek har son nævnt tidligere i sagen, fejl i farverne på vores reserveringsknapper

#4 Updated by Gitte Barlach almost 5 years ago

  • Target version changed from DDB CMS 2015 3. opgradering to DDB CMS 2015 4. opgradering

#5 Updated by Rolf Madsen almost 5 years ago

  • Target version changed from DDB CMS 2015 4. opgradering to 36

#6 Updated by Martin Cording over 4 years ago

We were not able to reproduce this issue.

#7 Updated by Gitte Barlach over 4 years ago

  • Status changed from Open (waiting) to Need more info
  • Assignee set to Rolf Madsen

#8 Updated by Rolf Madsen over 4 years ago

  • Assignee changed from Rolf Madsen to Martin Cording

@Martin, er det her et issue hvor vi skal have DBC til at kigge i databasen for at få hul på problemet?

#9 Updated by Martin Cording over 4 years ago

  • Assignee changed from Martin Cording to Rolf Madsen

The required resources to solve this problem is transverse, and I do not think that it will be beneficial to let DBC troubleshoot the issue.
One thing is to get a dump of an instance in order to reproduce it, and another is to frontend-wise debug why the issue happens - and commit the fix to code for review.

#10 Updated by Rolf Madsen over 4 years ago

  • Assignee changed from Rolf Madsen to Martin Cording

Så er det mest hensigsmæssige så at jeg beder DBC om at lave et DB dump af et af de biblioteker der har problemet og så lade jer analysere videre på det?

#11 Updated by Martin Cording over 4 years ago

  • Assignee changed from Martin Cording to Rolf Madsen

Yea, either us or anyone else with frontend resources able to debug this!

#12 Updated by Rolf Madsen over 4 years ago

  • Target version changed from 36 to 39

#13 Updated by Rolf Madsen over 4 years ago

  • Assignee changed from Rolf Madsen to Martin Cording

#14 Updated by Martin Cording over 4 years ago

  • Assignee changed from Martin Cording to Rolf Madsen

Please assign this back to me, when a DB dump (and a backup of the sites/default/color/ directory) is available.

#15 Updated by Rolf Madsen over 4 years ago

  • Assignee changed from Rolf Madsen to Christian Vandel

Jeg har oprettet en sag hos DBC og bedt dem lave et databasedump til jer.

#16 Updated by Rolf Madsen over 4 years ago

Din sag har fået nummer: 27164

#17 Updated by Martin Cording over 4 years ago

@Christian: When can we expect this dump of code+DB?

#18 Updated by Rolf Madsen over 4 years ago

  • Status changed from Need more info to Ready for development
  • Assignee changed from Christian Vandel to Martin Cording

#19 Updated by Martin Cording over 4 years ago

We have not been able to reproduce this in time, due to late delivery of dump from DBC.

This will be included in our next development iteration.

#20 Updated by Rolf Madsen over 4 years ago

  • Target version changed from 39 to DDB CMS 2016 1. opgradering

#21 Updated by Rolf Madsen over 4 years ago

  • Target version changed from DDB CMS 2016 1. opgradering to 42

#22 Updated by Rolf Madsen over 4 years ago

  • Assignee changed from Martin Cording to Laura Holm

#23 Updated by Rolf Madsen over 4 years ago

  • Assignee deleted (Laura Holm)

#24 Updated by Rolf Madsen over 4 years ago

  • Target version changed from 42 to DDB CMS 2016 1. opgradering

#25 Updated by Rolf Madsen over 4 years ago

  • Target version changed from DDB CMS 2016 1. opgradering to DDB CMS 2016 2. opgradering (Inlead sprintbacklog)

#26 Updated by Martin Cording about 4 years ago

  • Status changed from Ready for development to Need more info
  • Assignee set to Anonymous

Vi skal bruge et full-site dump uden symlinks.

#27 Updated by Martin Cording about 4 years ago

  • Target version changed from DDB CMS 2016 2. opgradering (Inlead sprintbacklog) to DDB CMS 2016 2. opgradering

#28 Updated by Rolf Madsen about 4 years ago

  • Target version changed from DDB CMS 2016 2. opgradering to DDB CMS 2016 2. opgradering (DBC sprintbacklog)

#29 Updated by Anonymous about 4 years ago

I "Dropbox/DDB CMS Tarball/ previous releases" ligger nu en tarball med et Aegir-dump af Vallensbæks prod-site (vallensbaek.ddbcms.dk-20160314.....) samt DDB-pakken: ding2-7.x-2.4.4.tar.gz som tilsammen udgør et komplet Vallensbæk-site.
Når vallensbaek.ddbcms.dk-20160314.... er hentet bedes den venligst slettet.

#30 Updated by Rolf Madsen about 4 years ago

  • Assignee changed from Anonymous to Laura Holm

#31 Updated by Rolf Madsen about 4 years ago

  • Status changed from Need more info to Ready for development

#32 Updated by per johansen about 4 years ago

  • Status changed from Ready for development to Development
  • Assignee changed from Laura Holm to per johansen
  • Kategorier Søgning - Materialevisning added

#33 Updated by per johansen about 4 years ago

  • Status changed from Development to Need more info
  • Assignee changed from per johansen to Betina Nakel Thorsager

Jeg har brugt nogle timer på at genskabe den meget flotte knap fra allerød - Det kunne jeg ikke.

Jeg kunne godt tænke mig at vide hvordan den er blvet så flot i første omgang, for lige nøjagtig reservations knappen er en af de knapper man IKKE kan konfigurere i colors.

Måske Betina eller Svend kan fortælle mig hvad de har gjort ??

#34 Updated by per johansen about 4 years ago

okay nu blev jeg alligevel stædig, og med lidt hjælp fra Jørgen (og tilfældighederne) fandt vi ud af det - jeg prøver at forklare.

Der er defineret en række farver, som kan konfigureres. Når farveeditoren åbnes sættes første farve som den der er valgt (skal redigeres). Når der vælges en farve i paletten ændres den valgte farves værdi.

Så er der bare det at TO af felterne er skjult vha css :
.form-item-palette-base {
display: none;
}

.form-item-palette-text {
display: none;
}

Og det er netop base der er det første i paletten. DVS at hvis man åbner farveeditoren, leger lidt med paletten UDEN at vælge en farve der skal redigeres, og trykker gem SÅ ÆNDRES BASE FARVEN.

Der er jo nok en grund til at base og text bliver skjult, men umiddelbart er den nemmeste løsning at vise dem igen.

Done a pull-request - tell me if there's a better solution
https://github.com/ding2/ding2/pull/212

#35 Updated by per johansen about 4 years ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Betina Nakel Thorsager to Kasper Garnæs

#36 Updated by Kasper Garnæs about 4 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to per johansen

Det ligner du mangler at få nogle commits med, Per. Der er ingen reelle ændringer i pull requestet.

#37 Updated by per johansen almost 4 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from per johansen to Kasper Garnæs

jeg har opdateret pull-requesten

#38 Updated by Gitte Barlach almost 4 years ago

  • Assignee changed from Kasper Garnæs to Jesper Kristensen

#39 Updated by Jesper Kristensen almost 4 years ago

  • Assignee changed from Jesper Kristensen to Gitte Barlach

Koden ser fin ud, men jeg under mig over hvorfor det har være sat til display none.

@GBA: kunne vi spørge Brian/Yde omkring dette ?

Ellers er coden reviewed.

#40 Updated by Gitte Barlach almost 4 years ago

Fra Brian:

Jeg kan ikke forklare hvorfor den er display: none, men jeg tænker at der er nogen der har tænkt det ikke skal kunne ændres på via theme settings (er det ikke her de snakker om?).
Der er så nok bare ikke taget højde for at selvom der er display none så gemmer den værdierne alligevel.

#41 Updated by Gitte Barlach almost 4 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#42 Updated by Jesper Kristensen almost 4 years ago

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

Afventer næste release

#43 Updated by Jesper Kristensen almost 4 years ago

  • Status changed from Reviewed to Technical test

#44 Updated by Rolf Madsen almost 4 years ago

  • Target version changed from DDB CMS 2016 2. opgradering (DBC sprintbacklog) to DDB CMS 2016 2. opgradering

#45 Updated by Rolf Madsen over 3 years ago

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

Testet og godkendt.

Jeg har teste via følgende fremgangsmåde:

  1. åbnet https://vanilla-alma.ddbcms.dk/admin/appearance/settings/ddbasic.
  2. valgt en farve i paletten uden at vælge et felt.
  3. observeret at Base color ændrer farve.
  4. Trykket på Gem knappen.
  5. Åbnet et materiale med availability "Available; Bog"
  6. Verificeret at Reservér knappen ikke har samme farve som base color er sat til

#46 Updated by Lotte Tøstesen about 3 years ago

Denne fejl har vi igen på grevebibliotek.dk efter opgradering til 3.0.2. Farven slår også igennem på kvitterings-pop-up'en når man tilføjer et materiale til en liste samt boksen der vises ved log-ud pga. inaktivitet. Den eneste farve vi har ændret er den nye Grundfarve for at skifte farve på interaktionsboksen. Vil dog tro, at fejlen bliver irrelevant med udrulning af det nye tema, hvor der ikke er så mange farveindstillinger - jeg har ikke kunnet genskabe fejlen på vores nyeste testsite (rc0).

#47 Updated by Simon Holt about 3 years ago

Hej Lotte. Du skrev det her på Facebook:

https://www.facebook.com/groups/ddbcms/permalink/521255818049412/

Så I har valgt at bruge base-farven til at farve ding_interaction pane. I http://platform.dandigbib.org/issues/1229#note-34 i denne sag fandt vi ud af, at det var denne base-farve der var skyld i det.

Så måske er problemet igen at base-farven ændres?

#48 Updated by Lotte Tøstesen about 3 years ago

Ja, for jeg har valgt en dejlig grøn som basefarve, ikke den der brune som jeg har set før på andre bibliotekers reserverknapper. Men dvs at hvis man ikke gider den blå, må man leve med den bruge. fedt :-S

#49 Updated by Simon Holt about 3 years ago

Måske kan der defineres en ny farve i color-paletten til ding_interaction panes?

#50 Updated by Lotte Tøstesen about 3 years ago

I første omgang er løsningen nok bare ikke at ændre Grundfarven.
Og i det nye tema er reserverknappen altid sort, så jeg tror ikke vi får samme problem fra næste opdatering.

Also available in: Atom PDF