FNNDSC / ChRIS_ui

UI for ChRIS
MIT License
53 stars 115 forks source link

(CHRIS UI - P1 - DESIGN) Redesign of the feed details layout #39

Open mairin opened 5 years ago

mairin commented 5 years ago

This was discussed at the UI meeting @BSS in late June.

There is some confusion about how the data in the blue column next to the graph vs. the data in the header bar and the white content area below the graph relate to each other in the feed details page.

Shania's pipeline design uses the header and white content area below only for pipeline-wide details, and only node-specific content (related to the highlighted node) appear in the blue column next to the graph.

The feed design needs to be cleaned up to follow this pattern because the lower white content area is more mixed up than this. One wishlist item we discussed was an embedded output viewer in the content area!

mairin commented 5 years ago

So this was the first redesign attempted a while back. It makes the lower content area solely focused on node details which I think now is the wrong way, but posting here for historical context.

feed-details_right-panel-revision_1

mairin commented 5 years ago

Okay here's a refresh of the refresh design. :-) Some features to look for:

feed-details_right-panel-revision_2

mairin commented 5 years ago

Here's an update based on @jcaianirh 's feedback this morning. Changelog:

feed-details_right-panel-revision_3

mairin commented 5 years ago

Two pending changes based on discussion in Slack just now:

mairin commented 5 years ago

OK here is a design I think that's in good shape to start working on implementing (have a little thumb of the selected node details panel to the right to show what it looks like under the popup): feed-details_right-panel-revision_5

The idea with the copy/paste paste popup, is you could shift+click the filename to access it. On mobile, these lists degrade into cards, right? In which case, for mobile sizes only, the copy/paste text area could be displayed along the bottom of each card.

mairin commented 5 years ago

One stupid issue - this design doesn't display the feed description. 🤦 I wouldn't consider this an MVP blocker. I'd simply add it as another field in the top header. I can update the mockup accordingly, just a head's up!

mairin commented 5 years ago

At the UI standup yesterday afternoon, we agreed on some changes to this design: