go-spatial / fresco

An open source vector tile style editor
https://fresco.go-spatial.org
Apache License 2.0
234 stars 26 forks source link

ID change requires separate click per character #52

Closed PetersonGIS closed 4 years ago

PetersonGIS commented 6 years ago

Editing an id requires you to click inside the id quotes for each letter.

justenPalmer commented 6 years ago

@PetersonGIS, while it is possible to change a layer ID through the JSON editor, it's difficult to support because of the way the data syncs with every change and has to re-navigate the user back to the page (the layer ID is used in the page path). However, there is another supported way of changing the ID which works seamlessly. Just click on the layer ID in the top bar and it will change to input field where you can change the ID. Because the change is only propagated when you press enter, it doesn't mess with the user experience.

screen shot 2018-06-11 at 3 38 22 am

I need to document this functionality and figure out a way of relating the ID should not be changed in the JSON directly.

PetersonGIS commented 6 years ago

Ok, didn't realize you could change it up there. Good to know.

Definitely something to make other users aware of somehow. Thanks.

justenPalmer commented 4 years ago

ID changes should no longer cause any issues on dev.