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

Input/result tab contents are confusing #197

Closed bmmpxf closed 9 years ago

bmmpxf commented 9 years ago

I used the geo:area process, gave it a WKT of POLYGON((0 0, 1 0, 1 1, 0 1, 0 0)) and then Ran the process.

Everything went fine, but the tabs confused me. The Input tab has different content depending on what's clicked, but the Results tab is always the same. What's the purpose of having an Input tab on the geo:area block or the result block? Why not show the contents of the Result tab only when you click the result? That way you could get rid of the tabs outright.

If this becomes more obvious with more complex processes, I'll update with my findings.

bartvde commented 9 years ago

There is no purpose for the input tab in those occasions and that's why the content is cleared actually.

The results tab gets activated when the response comes in. Otherwise it might not be obvious to the end user when the result is ready?

For the result tab we had thought about showing the result of the clicked process but this never materialised, and it only shows the output of the latest run process (just like the XML view does).

The separation comes from the wireframes so looping in @eshon here as well.

eshon commented 9 years ago

My brain's a little fried this week so I asked Rollie. Here's what would be ideal:

geo:area: 1.0 geo:crosses: false ....

That way it's clear the Results tab is not meant to be tied to any particular Result node.

Is that feasible?

bartvde commented 9 years ago

Never delete contents in Results is not going to be feasible for this round.

bartvde commented 9 years ago

I am going to add the process abstract like Rolando suggested, but that's as far as the changes go for this ticket in the current round. So milestone FUTURE for the rest.

eshon commented 9 years ago

Please rename "Results" to "Console" then to reduce confusion.

bartvde commented 9 years ago

Okay

bartvde commented 9 years ago

cc @sbalasub as this will influence some of the screenshots

cc @bmmpxf since this involves doc changes as well then

bmmpxf commented 9 years ago

I don't think I'm totally following the discussion, but as far as my part is concerned, are you talking about the title of the tab changing or the entire right-side panel (which I have been calling the I/O panel)? I may need to redo a few screenshots as well, so please let me know when that part has been done and redeployed.

bartvde commented 9 years ago

only rename the Results tab to Console, other changes have too many implications now, I'll upload soon

bartvde commented 9 years ago

@bmmpxf updated server just now

bmmpxf commented 9 years ago

Makes sense, screenshots updated.

bartvde commented 9 years ago

@bmmpxf there is also some text which needs updating right? Or you're on that already?

Results are either shown on the Results tab

bmmpxf commented 9 years ago

Ah thanks, missed that one. It's updated now.