Project

General

Profile

Bug #3350

Søgeresultatsider for redaktionelt indhold fremstår ufærdige

Added by Nino Tiainen over 2 years ago. Updated over 1 year ago.

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

Description

Bl.a.

  • Forskellige typer af indhold er i øjeblikket ikke formateret identisk på søgeresultatsiderne. Se vedhæftede screendump fra aakb.dk hvor billede på side-teaser fylder stor del af skærmen. Andre indholdstyper vises ikke med billede. 
  • Umiddelbart er det svært at gennemskue hvad der er hvad. Er der tale om et arrangement, en nyhed eller noget helt andet.  

 

soegning.png (280 KB) soegning.png Nino Tiainen, 02/28/2018 05:19 PM
Skærmbillede 2018-03-08 08.23.45.png (41.6 KB) Skærmbillede 2018-03-08 08.23.45.png Nyhed i søgeresultatet Gitte Barlach, 03/08/2018 08:38 AM
Skærmbillede 2018-03-08 08.24.02.png (22.6 KB) Skærmbillede 2018-03-08 08.24.02.png Arrangement i søgeresultatet. Det fremgår dog ikke nogen steder før man klikker og kommer ind på det Gitte Barlach, 03/08/2018 08:39 AM
Skærmbillede 2018-03-08 08.29.52.png (297 KB) Skærmbillede 2018-03-08 08.29.52.png Arrangement i søgeresultatet. Billedstørrelsen er forkert Gitte Barlach, 03/08/2018 08:40 AM
Skærmbillede 2018-03-08 08.34.16.png (192 KB) Skærmbillede 2018-03-08 08.34.16.png Indholdstypen Staff i søgeresultatet - her er noget galt med stylingen Gitte Barlach, 03/08/2018 08:41 AM
Skærmbillede 2018-09-03 09.54.48.png (18.3 KB) Skærmbillede 2018-09-03 09.54.48.png Gitte Barlach, 09/03/2018 10:01 AM
Skærmbillede 2018-09-03 09.53.46.png (64.2 KB) Skærmbillede 2018-09-03 09.53.46.png Gitte Barlach, 09/03/2018 10:01 AM
Skærmbillede 2018-09-03 09.54.23.png (55.1 KB) Skærmbillede 2018-09-03 09.54.23.png Gitte Barlach, 09/03/2018 10:01 AM
Skærmbillede 2018-09-03 09.55.14.png (19.7 KB) Skærmbillede 2018-09-03 09.55.14.png Gitte Barlach, 09/03/2018 10:01 AM

History

#1 Updated by Rolf Madsen over 2 years ago

  • Status changed from New to Needs analysis
  • Target version set to Release 28 - Søge abstraktionslag (SAL) - (7.x-4.3.1)

Visning af hjemmeside søgeresultatet fejler pt. så jeg kan ikke testet og analysere dette issue.

#2 Updated by Gitte Barlach over 2 years ago

Vedlagte skærmbilleder (fra aakb.dk med DDB CMS release 27) illustrerer problematikken:

Jeg søger på "krimi" og vælger "Resultater fra hjemmesiden". Der er 37 hits.

 

De første 2 poster, samt fjerde og femte post i søgeresultatet er nyheder.

Den tredie post og sjette post er et arrangement. Men det kan man rent faktisk ikke se, før man klikker på det og kommer til selve visningen af arrangementet. Der er endvidere noget galt med stylingen/billedestørrelsen. 

Stylignen er endvidere også forkert når man har søgt på ansat  på bibliotekerne.  

Konklusionen må derfor være: Alle indholdstyperne bør gennemgås og styles mhp god UX og visning i søgeresultatoversigten

#3 Updated by Christel Krabbenhøft over 2 years ago

  • Subject changed from Søgeresultatsider for redaktionelt indhold fremstår ufærdige. to Søgeresultatsider for redaktionelt indhold fremstår ufærdige
  • Status changed from Needs analysis to Ready for development
  • Assignee changed from Rolf Madsen to Anonymous
  • Target version changed from Release 28 - Søge abstraktionslag (SAL) - (7.x-4.3.1) to Release 29-2 - Bugfixes (B14)

#4 Updated by Anonymous over 2 years ago

  • Assignee changed from Anonymous to Anonymous

#5 Updated by Anonymous over 2 years ago

  • Assignee changed from Anonymous to Anonymous

#6 Updated by Anonymous over 2 years ago

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

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

Vi har været nødt til at indkludere et patch i ding2.make filen for at komme en bug til livs, der er opstået efter features modulet er blevet opdateret. Buggen sletter eksisterende grupper når der feature-ekspores.

Det er altså en generel ting, og gør sig gældende hver gang vi feature-eksportere, så det ville være godt, hvis patch'et kan merges i core så hurtigt som muligt. Enten gennem dette PR eller ved at oprette et selvstændigt issue.

Samme problem gør sig gældende med pages, hvor nogle sider slettes ved feature-eksport. Så kan vi ikke også få et issue til denne bug?

#7 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#8 Updated by Jesper Kristensen over 2 years ago

  • Assignee changed from Jesper Kristensen to Anonymous

Hej Kristian

Ændringer ser fine ud.

Jeg vil bare lige sikker mig at de ændringer features laver i forhold til adaptive images og image style er rigtige? Kan du verify at det er korret at disse ændringer skal med?

#9 Updated by Anonymous over 2 years ago

  • Assignee changed from Anonymous to Jesper Kristensen

Hej Jesper

Ja - jeg kan verificere, at det er korrekt.

De fleste ændringer skyldes at features modulet er blevet opdateret.

 

Venlig hilsen

Kristian

 

#10 Updated by Jørgen Nielsen over 2 years ago

  • Assignee changed from Jesper Kristensen to Jørgen Nielsen

#11 Updated by Jørgen Nielsen over 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Anonymous

Der er en merge-konflikt og et par Scrutinizer fejl, der skal tages hånd om

#12 Updated by Anonymous over 2 years ago

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

PR opdateret

#13 Updated by Gitte Barlach over 2 years ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#14 Updated by Jørgen Nielsen over 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Anonymous

Det ser fint ud - men jeg har lige et spørgsmål

#15 Updated by Anonymous over 2 years ago

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

Jeg har svaret på spørgsmål på PR.

#16 Updated by Jørgen Nielsen over 2 years ago

  • Status changed from Needs code review to Reviewed

reviewet og godkendt

#17 Updated by Kasper Garnæs about 2 years ago

  • Status changed from Reviewed to Technical test

Merged.

#18 Updated by Gitte Barlach about 2 years ago

Testet på vanilla-29 med 4.5.0-rc1

Med forbehold for at jeg ikke har kunnet teste hvordan en personaleprofil vises i søgeresultatet, er de andre indholdstyper nu stylet, så søgeresultatet ser ensartet ud samt der er en "Læs mere"-knap. Man kan ligeledes skelne mellem indholdstyperne i søgeresultatet. Se vedlagte skærmdumps.

Det eneste der virker uhensigtsmæssigt er at arrangementer hedder "Begivenhed" og temaer hedder "Grupper"
Det er teknisk set helt logisk, men ikke i overensstmmelse med det sprogbrug vi anvender overfor slutbrugeren på sitet. Her hedder det jo arrangementer og tema´er. 
Kan det rettes så vi også i søgeresultatet bruger de to betegnelser??

#19 Updated by Anonymous about 2 years ago

  • Assignee changed from Anonymous to Anonymous

#20 Updated by Anonymous about 2 years ago

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

#21 Updated by Jesper Kristensen about 2 years ago

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

Løsningen skal kunne oversættelses ingen dansk strings i koden.

#22 Updated by Anonymous about 2 years ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Anonymous to Jesper Kristensen

Hej Jesper. Det er modtaget. Vil gerne høre hvilken løsning der foretrækkes her:

1. Vi oversætter generelt (via translate interface) strengen 'Groups' med 'Temaer', og 'Event' med 'Arrangement' (istedet for hhv. Grupper og Begivenhed).

2. Vi gør som ovenstående PR, men switcher på ikke oversat node_type, og kører det igennem en t() funktion, men med en context (search result).

#23 Updated by Rolf Madsen almost 2 years ago

  • Status changed from Need more info to Needs code review

#24 Updated by Kasper Garnæs almost 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jesper Kristensen to Anonymous

Navngivningen af indholdstyperne bør være konsekvent på tværs af sitet så hvis "group" hedder "tema" og "event" hedder "arrangement" på dansk så bør de gøre det på tværs af sitet. Som jeg ser det er oversættelsesmodulet den bedste måde vi har at sikre dette.

#25 Updated by Anonymous almost 2 years ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Anonymous to Kasper Garnæs

Hej Kasper.

Vi er lidt uklare på proceduren ift. at få nye globale strenge med i oversættelser? Kan du/i guide os, eventuelt referere til et commit hvor det er gjort før.

Forsøgte at opdatere de relevante strenge i da.po filen i ding2/translations, indsætte en ding_update_[number] function i ding2.install som kører ding2_translation_update() - og derefter køre en drush updb, men uden held.

På forhånd tak.

#26 Updated by Rolf Madsen almost 2 years ago

  • Status changed from Need more info to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Anonymous

Hej Steen

Jeg plejer at eksportere strengene og indsætte dem manuelt i en masteroversættelse som så tilføjes nye versioner.

Det vil være en stor hjælp, hvis I kan indsætte dem i sagen ala:

#: /index.php
msgid "Ting openlist importer"
msgstr "Openlist importer"

Så slipper jeg for at initiere dem for at jeg kan eksportere dem.

#27 Updated by Anonymous almost 2 years ago

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

#28 Updated by Anonymous almost 2 years ago

  • Assignee changed from Anonymous to Rolf Madsen

#29 Updated by Christel Krabbenhøft almost 2 years ago

  • Assignee changed from Rolf Madsen to Gitte Barlach

#30 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#31 Updated by Rolf Madsen over 1 year ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach
  • Target version changed from Release 29-2 - Bugfixes (B14) to Release 31 - bugfixes

Dette issue flyttes til Release 31 med henblik på review.

#32 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#33 Updated by Jørgen Nielsen over 1 year ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Anonymous

der er stadigvæk et udestående ifm. en manglende update hook. Se: ding2.install

#34 Updated by Anonymous over 1 year ago

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

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

  • Assignee changed from Christel Krabbenhøft to Gitte Barlach

#36 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#37 Updated by Jørgen Nielsen over 1 year ago

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

reviewet og godkendt

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

  • Status changed from Reviewed to Technical test
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

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

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#40 Updated by Gitte Barlach over 1 year ago

Jeg tester denne.

#41 Updated by Gitte Barlach over 1 year ago

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

Testet på vanilla-fbs med 7.x-4.8.0-rc1
Har verificeret seneste rettelse i sagen, nemlig at 'grupper' nu er oversat til 'tema' og 'event' til 'arramgement' i søgeresultatoversigten https://vanilla-fbs.ddbcms.dk/search/node/xx

Hermed godkendt.

Also available in: Atom PDF