Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Selecteer bescherming #12

Closed
claeyswo opened this issue Apr 2, 2015 · 18 comments · Fixed by #38
Closed

Selecteer bescherming #12

claeyswo opened this issue Apr 2, 2015 · 18 comments · Fixed by #38
Assignees
Milestone

Comments

@claeyswo
Copy link
Member

claeyswo commented Apr 2, 2015

Net zoals er nu een knop is selecteer perceel moet er een knop bijkomen selecteer bescherming die de afbakening van de beschermde objecten overneemt.

@dieuska
Copy link

dieuska commented Jun 9, 2015

Mockup voor zoneerder
(Enkel opdeling, geen layout)

zoneerder_wireframe 3
F= Flash
Z=Zoom
D= Delete

@cedrikv
Copy link
Contributor

cedrikv commented Jul 1, 2015

Het overzichtslijstje is wel enkel mogelijk tijdens het intekenen, na opslaan is er maar 1 zone (alle info ivm bvb de onderliggende percelen wordt niet mee opgeslagen). Er blijft dus enkel een multipolygoon over.
Bij editeren van bestaande zone is het dus niet mogelijk om een 'deelzone' te gaan flashen, deleten,...

@cedrikv
Copy link
Contributor

cedrikv commented Jul 3, 2015

Tussentijdse versie:
zoneerder

Hier al opmerkingen over?

@BartSaelen
Copy link

ziet er al veel beter uit dan de huidige versie

@dieuska
Copy link

dieuska commented Jul 3, 2015

ziet er goed uit!

enkele opmerkingen:
*Typo in Zone: polyogoon

  • Bij toevoegen WKT: Als ze de string erin hebben geplakt, is er dan een button die deze polygoon effectief aan de Zone-lijst toevoegt?
    -(Mss ervoor een PLUS- icoon voor WKT zetten, waarop geduwd kan worden om de WKT-string effectief aan de lijst toe te voegen? Of is de tekst 'WKT' momenteel clickable voor de toevoeging effectief uit te voeren?

@koenedaele
Copy link
Member

Ik mis nog ergens een ninja. Of een dinosaurus. Of een ninja op een dinosaurus.

@cedrikv
Copy link
Contributor

cedrikv commented Jul 3, 2015

Zoiets?

ninjadino

@cedrikv
Copy link
Contributor

cedrikv commented Jul 3, 2015

of is de boog er wa over?

@cedrikv
Copy link
Contributor

cedrikv commented Jul 3, 2015

@karel: WKT voor het inputveld is inderdaad de knop die je zoekt, maar dan nog ongestijld (zal zoals in je mockup worden).

@koenedaele
Copy link
Member

Vervang de boog door shuriken en 't is perfect!

@claeyswo claeyswo added this to the 0.3.0 milestone Jul 7, 2015
@cedrikv
Copy link
Contributor

cedrikv commented Jul 8, 2015

Ik ben niet helemaal zeker welke logica er nog achter de 'bewaar' en annuleren' knoppen moet zitten, als je ervan uit gaat dat de zoneerder in een andere app ingebed wordt:
-In principe wordt een zone bewaard op het niveau van de app, bvb adviezen. de zone extra moeten bewaren kan verwarrend zijn. hetzelfde voor annuleren.
-bij annuleren is er nog een extra use case dat je je huidige teken of selecteer actie wilt apart annuleren, zonder al je vorige edits teniet te doen

@dieuska
Copy link

dieuska commented Jul 8, 2015

Bewaren en annuleren zijn idd verwarrende namen indien dit ingebed zit in de app waar je de effectieve 'save' uitvoert.

Toch lijkt mij mss handig om hier 2 buttons te hebben (met een betere naam):
*button 1: Ik wil het zoneren afsluiten en ben akkoord met mijn zonering.
*button 2: ik wil het zoneren afsluiten en ge moogt al mijn acties op de zoneringspagina negeren

Wat zou een betere benaming zijn?
button 1: OK
button 2: Annuleer/Negeer/Sluit af

Ofwel houden we het simpel en voorzien we enkel één button om het zoneringsvenster af te sluiten
(en indien gewenst zetten we ergens anders op de pagina een 'reset' button om naar de oorspronkelijke zonering te gaan...)

@cedrikv
Copy link
Contributor

cedrikv commented Jul 8, 2015

Ik vrees voor verwarring bij de gebruikers. In de andere tabs van de beheer interfaces (relaties, bestanden,..) moet er niet apart op 'ok' geklikt worden. Je hebt de algemene 'opslaan' en 'annuleren', die dan voor alle tabs gelden.
Wat gebeurt er bvb als iemand niet klikt op button 1? gaan al zijn bewerkingen verloren als het beheersplan opgeslagen wordt?

Je kan het zoneren ook niet 'afsluiten', zolang het beheersplan in de editeer mode zit zit zal dit ook zo zijn voor de zoneerder.

Een aparte annuleer knop kan nog nuttig zijn, maar in de andere tabs is dit ook niet. Dankzij het extra grid met alle zones kan je ook gewoon zelf je aanpassingen terug verwijderen.

@claeyswo
Copy link
Member Author

claeyswo commented Jul 8, 2015

+1 om die knoppen gewoon te verwijderen

@dieuska
Copy link

dieuska commented Jul 8, 2015

idd hoe minder knoppen hoe beter, dus als alle cases zijn opgevangen voor de gebruiker door de algemene knoppen van de applicatie, des te beter.

(een herstel/reset-knop is mss een nice to have, maar lijkt me enkel nuttig indien dit op vraag van de gebruikers komt en ze dit dus effectief nodig hebben)

@claeyswo
Copy link
Member Author

claeyswo commented Aug 6, 2015

De verschillende lagen voor de beschermingen:

vioe_geoportaal:beschermde_landschappen
vioe_geoportaal:beschermde_dorps_en_stadsgezichten
vioe_geoportaal:beschermde_archeologische_zones
vioe_geoportaal:beschermde_monumenten

Ik denk niet dat het hier nodig is om geïnventariseerd erfgoed te kunnen selecteren. Enkel de koppeling moet kunnen gelegd worden.

@koenedaele De vraag van @cedrikv was of het niet mogelijk is de 4 bovenstaande lagen te bevragen met 1 call?

@vanbosa
Copy link
Contributor

vanbosa commented Aug 6, 2015

vioe:beschermde_landschappen
vioe:beschermde_archeologische_sites
vioe:beschermde_monumenten
vioe:beschermde_dorps_en_stadsgezichten

geoportaal lagen geven geen data weer.

Volgende week is zien als dev geo geoserver actief is
dev-geo geoserver opzetten: ogc script, vervangen door tomcat instellingen/settings
beschermingen rechtstreeks doen ipv porxy script.

wfs bevragen en werken met views.
outputFormat json ipv van default xml.

@cedrikv
Copy link
Contributor

cedrikv commented Aug 7, 2015

Note van Koen: check code in geoportaal voro wfs calls (ivm lagen combineren & stijl)

@cedrikv cedrikv mentioned this issue Aug 12, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants