Closed niekh1234 closed 1 year ago
Lijkt mij een prima toevoeging! Ik heb nog wel een paar puntjes voordat we dit gaan mergen.
Het is nu mogelijk om een niet bestaande organisatie in te voeren. Kunnen we dat aan de voorkant (en achterkant) voorkomen?
Het lijkt er op dat de suggesties een exacte match nodig hebben. "sud" levert niks op, alleen als ik "súd" in type zie ik Súdwest-Fryslân naar voren komen. Kunnen we deze aanpassen zodat 'ie velden met accenten/diakrieten ook matcht met het basis karakter? (ü -> u, â -> a, é -> e, etc.)
Er moet nog iets van een instructie label bij, zodat gebruikers weten dat het veld automatisch suggesties geeft.
Ik betwijfel of er een mooie oplossing is met die speciale tekens (ü -> u, â -> a, é -> e, etc.) zonder een hele hoop javascript.
Misschien dan toch maar laten zo.
Ik ben trouwens al aardig op weg met een aantal aanpassingen op de plugin die voor ons handig gaan zijn welke wellicht teveel afwijken van jouw originele filosofie voor de package.
Het gaat vooral om het volgende: we willen ipv REST endpoints alles in elasticsearch hebben, verder willen we de zoekdienst 2.0 gebruiken voor de bekendmakingen.
Dan verder heb ik een aantal nice-to-haves toegevoegd zoals de mogelijkheid om de waarde van xml attributen te gebruiken.
Sta jij er open voor om dit aan de package toe te voegen? Zie mijn branch https://github.com/niekh1234/open-government-publications/commits/chore/prod-ready . Ik heb mijn best gedaan om het een beetje clean te houden maar ik zal je zeggen dat ik niet al te bekend ben met vanilla WP plugins.
Hi Sander, beetje all-over-the place maar dit is niet meer nodig.
Ik ben een beetje met de plugin aan het experimenteren en ik moet zeggen petje af. Hier alvast een idee: ipv een select zo'n input+datalist gebruiken, scheelt weer scrollen door die lange lijst. Wanneer je typt krijg je "suggesties" met de mogelijke opties.
Wat denk jij?