planetfederal / wps-gui

Graphical User Interface (Model Builder) for OGC:WPS (Web Processing Services)
http://boundlessgeo.com
Apache License 2.0
26 stars 15 forks source link

What does File / Open do? #202

Closed bmmpxf closed 9 years ago

bmmpxf commented 9 years ago

There isn't any result to these options as far as I can tell. If either are supposed to bring up a dialog, they don't.

bartvde commented 9 years ago

No they save your current state to the browser's Local Storage. When you come back the next day, you can press Open and your last saved state will come back from the browser's Local Storage.

Maybe we should add a dialog with this info after Save completes?

bartvde commented 9 years ago

I am guessing the dialog will be annoying for people using this function often. Making the options more explicit in their wording.

@bmmpxf can you add this to the docs as well?

bmmpxf commented 9 years ago

I'm not certain that this is any less confusing. Perhaps I'm not the target audience, but when I read "Local Storage" I think of a file download, not a browser cache. Is there another phrase we can use?

And by the way, might want to make the width of the menu bigger:

localstorage

bartvde commented 9 years ago

So what wording do you suggest? Or what other change?

bmmpxf commented 9 years ago

Hmm, I'd say "Local browser storage" or "Browser cache" or something along those lines. Something to make it clear that no files are being transferred.

And while a popup may be annoying, if there were some way, perhaps the bottom left or right of the screen, to say "Saved to {whatever we call it}", that would give the user some helpful feedback.

bartvde commented 9 years ago

So maybe going back to the old menu titles and providing the user feedback that disappears after X seconds?

bmmpxf commented 9 years ago

I'd prefer to keep the more descriptive info in both places if possible.

bartvde commented 9 years ago

okay I'll work on something, I'll let you know when it's ready.

bartvde commented 9 years ago

@bmmpxf changes deployed let me know what you think

bmmpxf commented 9 years ago

Looks good, though I'd have some feedback for Open as well.

bartvde commented 9 years ago

with Open you can see it loaded in the canvas or not?

bartvde commented 9 years ago

but it's easy enough to add if you really want

bmmpxf commented 9 years ago

Open works fine. I know it might be redundant, but I'd still like to see "Configuration loaded from browser storage" or however you want to phrase it.

bartvde commented 9 years ago

Since there is not a lot of space I can add Loaded successfully if that makes sense.

bartvde commented 9 years ago

is that okay as the text @bmmpxf or you have another suggestion (that is short enough) ?

bmmpxf commented 9 years ago

That's fine, if space is an issue.