Omlægning til GSearch

[klavspc]

@AsgerPetersen : Giv den nogle tæv og se hvordan den opfører sig

• Hvis der er udeståender -> @klavspc til rettelser
• Hvis den er fin -> @MieWinstrup til finpudsning af tekster, release og nyhed

[klavspc]

• Når man holder en lille pause i indtastningen, så dropdownen udfyldes, så forsvinder fokus fra textfeltet. Hvis man er lidt for langsom til at indtaste, så er det virkeligt irriterende.

Det må jeg lige undersøge - Kan ikke replikere

• Netværksrequests optræder ikke i "Development tools". Måske hvis man bruger QgsNetworkAccessManager i stedet for QNetworkAccessManager? Det ser ud som om, Qgs-versionen arver fra Q-versionen:

Jeg prøver - Der logges nu

https://github.com/qgis/QGIS/blob/864fee0172ddd80438633e714591a0fadcf96de0/src/core/network/qgsnetworkaccessmanager.h#L269

• Skal det være "hus" for adresse (husnummer)?

Jeg er ikke sikker - man kan også søge på adresser i GSearch, hvad skulle det så i givet fald hedde?

• Max 3 resultater for hver kilde virker af meget lidt

Kan sagtens skrue op - det er opstået af default 25 pladser divideret med antal targets. Var det ikke default 25 i den gamle version også? - Har sat limit op til 40 (hard)

• Der er noget timingsproblem. Eksempel: jeg vil søge på "3kg". Jeg ser: ganske kort resultatet for "3kg" lynhurtigt afløst af resultatet for "3".

Jeps, kan sagtens fikses - Fikset

• Jeg er ikke vild med svensknøglen i panelet.

Helt væk eller tandhjul? - har lagt qgis's sædvanlige svensknøgle på som i

• Håndtering af NotAuthorized er holdt op med at virke. Det virker iøvrigt tilsyneladende heller ikke længere i den gamle version (funktion: https://github.com/Septima/qgis-geosearch/blob/6a29c1de7dfe9117150335c11e0efa3aff3a2caf/src/geosearch_dk/searchbox.py#L127 )

Kigger på det - men man får jo rudimentær info når man klikker på "Fejl" i listen. Det var den meddelelse jeg tænkte skulle gøres pænere

• Måske er der også et issue med at få ændrede settings til at træde i kraft.

Det har jeg kun set virke - jeg giver den nogle tæv igen - Kan ikke replikere

[klavspc]

@MieWinstrup : Jeg kigger på disse ting på mandag

[klavspc]

@AsgerPetersen Se mine kommentarer ovenfor (ændringer m/fed og kursiv)

[AsgerPetersen]

Cool!

Jeg synes, vi skal gå med "adr" for søgning i det, som langt de fleste kender som "adresser".

Jeg synes i øvrigt ikke, at præfixet ser ud til at fungere:

[klavspc]

@AsgerPetersen @MieWinstrup

• prefix-søgning fungerer
• "hus" er ført tilbage til "adr"
• Der er hover på settings-knappen

Jeg tænker, at den er ved at være klar til at Mie giver den nogle tæv

[MieWinstrup]

Hvad tænker I om at ændre indstillingens menuen fra det nuværende:

Til dette:

På QGIS-kurserne oplever jeg tit, at erfarne QGIS-brugere ikke kender muligheden for at filtrere i søgeresultaterne direkte fra søgefeltet. Og det giver ret god mening fx at indtaste adr: hvis man ved at det er en adresse man gerne vil finde. Så jeg kunne godt tænke mig at det stod et sted - andet end på github repo.

[AsgerPetersen]

Ja, god ide.

Hvad med

"Begræns søgning til følgende kommuner:" i stedet for "Kommunefilter (til at begrænse søgning til specifikke kommuner".

Og tilsvarende "Begræns søgning til resultater af disse typer:".

Den forklarende tekst synes jeg skal ind i boksen, som handler om filtrering på typer.

[MieWinstrup]

Fin justering, Asger! Jeg har rettet tekstdelen - men ved ikke hvordan jeg får flyttet den forklarende tekst ind i boksen (det giver dog god mening at gøre!)

[AsgerPetersen]

Jeg fikser lige tekstens placering

[AsgerPetersen]

Nu er gui opdateret. Der mangler oversættelse til engelsk.

[MieWinstrup]

Tak, @AsgerPetersen! Jeg har ikke fået oversættelsessetup til at virke på min windows maskine. Virker det hos dig? Kan jeg få dig til at lave oversættelsesfilen, så skal jeg nok oversætte den.

[MieWinstrup]

@AsgerPetersen - jeg har opdateret oversættelsesfilen. Kan du køre setuppet, så oversættelsen laves? Derefter tænker jeg at pluginet er klar til upload på QGIS plugin repository (det står jeg for)

[AsgerPetersen]

@MieWinstrup Den skulle være der nu. Check lige, at den er god hos dig også.