Project

General

Profile

Bug #5228

Søgeknap fejler på mobilbrowser

Added by Niels Kaagaard 4 months ago. Updated about 2 months ago.

Status:
Resolved (tag version)
Priority:
High
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Søgning - Søgefelt før søg - Brønd

Description

Søgefunktionen fungerer ikke på mobilbrowser, når man besøger undersider. Klik på luppen viser et kryds i stedet for en søgebjælke.

For eks: https://horsensbibliotek.dk/tags/selvbetjening 

Den vedhæftede video viser problemet. 

Jeg har afprøvet på Android og iPhone med Chrome og Safari browsere


Files

søgning.mp4 (5.32 MB) søgning.mp4 Niels Kaagaard, 08/25/2021 11:56 AM
søgefelt vises under wrappet hovedmenu.mp4 (1.42 MB) søgefelt vises under wrappet hovedmenu.mp4 Agnete Moos, 10/01/2021 11:23 AM
#1

Updated by Ninna Rasmussen 4 months ago

  • Assignee set to Árni Loftsson
  • Target version set to Release 34 - Bugfixes

Hej Arni. Vil du se på denne? 

 

 

#2

Updated by Ninna Rasmussen 4 months ago

  • Status changed from New to Ready for development
  • Priority changed from Normal to High
#3

Updated by Tue Gaston 4 months ago

Jeg kan reproducere problemet, når jeg prøver at søge fra https://horsensbibliotek.dk/tags/selvbetjening - men hvis jeg prøver fra https://horsensbibliotek.dk/ så virker det.

Nu gætter jeg bare, men det har formentlig noget at gøre med, at topmenuen bryder om og bliver i to linjer (med den skærmopløsning, min mobilskærm har, i hvert fald), når man er på https://horsensbibliotek.dk/tags/selvbetjening - mens den ikke gør på forsiden, fordi den dér har søgefeltet udvidet (hvilket så måske i virkeligheden er problemet).



Med andre ord tror jeg egentlig at søgefeltet er der, det er bare skjult under topmenuen.

#4

Updated by Árni Loftsson 4 months ago

Det gør jeg.

#5

Updated by Stefan Søndervang 3 months ago

Jeg tror problemet er at når man har valgt søgebaren 'Extended with profiles' under Search form style i /admin/config/ting/search, så f'er søgebaren på sider, som ikke er forsiden. Jeg testede genbib.dk, som også har valgt samme søgebar. Der er problemet det samme. På vores er der intet problem, men vi har også valgt 'Normal'. Ved ikke hvad der sker når man har valgt 'Extended'.

#6

Updated by Árni Loftsson 3 months ago

Fikset med PR: https://github.com/ding2/ding2/pull/1817

Det var rigtigt at problemet opstår hvis man har valgt søgeprofilen extended eller extended with profiles. 

#7

Updated by Árni Loftsson 3 months ago

  • Assignee changed from Árni Loftsson to Ninna Rasmussen
#8

Updated by Ninna Rasmussen 3 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Ninna Rasmussen to Gitte Barlach
#9

Updated by Gitte Barlach 3 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen
#10

Updated by Jørgen Nielsen 3 months ago

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

der er lige en kode standard rettelse, der bør fikses

#11

Updated by Árni Loftsson 3 months ago

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

Fikset

#12

Updated by Jørgen Nielsen 3 months ago

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

reviewet og godkendt

#13

Updated by Gitte Barlach 3 months ago

Testet på https://5228-search-button-pvd63ti-ton6gvqucinhw.eu-4.platformsh.site/

Testet i gængse browsere, herunder browser på mobil (iPhone)

Har verificeret at søgefeltet kan foldes ud på alle sider på sitet, når biblioteket har valgt søgeprofilen extended eller extended with profiles under /admin/config/ting/search, samt at sticky søgeprofil på udvalgt tema (studieunivers) stadig virker. 



Hermed videre til merge og integrationstest. 

#14

Updated by Kasper Garnæs 3 months ago

  • Status changed from Reviewed to Integrationstest (Versioner)
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

Der var en konflikt med andre ændringer. Jeg har forsøgt at resolve den på bedste vis.

#15

Updated by Kasper Garnæs 3 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach
#16

Updated by Agnete Moos 2 months ago

Testet på upgrade-fbs med 7.x-6.4.0-beta-1 med iPad i bredformat. Der er med vilje indsat mange hovedmenupunkter så menuen wrapper. Det er nemlig den case sagen omhandler. Ikke testet på Andriod.

Testet med de tre søgefeltsindstillinger: Normal, Extended og Extended with profiles

Testet som anonym og med indlogget bruger.

Det virker som det skal :-)

 

#17

Updated by Gitte Barlach about 2 months ago

  • Status changed from Integrationstest (Versioner) to Resolved (tag version)

Nu også testet på Android, og her virker  fold ind - fold ud af søgefeltet også. Hermed godkendt. 

Also available in: Atom PDF