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

Flyout isn't useful with long lists #1059

Open
Tracked by #1028
jawache opened this issue Oct 23, 2024 · 4 comments
Open
Tracked by #1028

Flyout isn't useful with long lists #1059

jawache opened this issue Oct 23, 2024 · 4 comments
Assignees
Milestone

Comments

@jawache
Copy link
Contributor

jawache commented Oct 23, 2024

@osamajandali I'm hovering over a pipeline plugin, but I can't see what outputs are highlighted because they are offscreen. The only way to see them is to scroll down which means i'm not hovering over the pipeline plugin anymore.

We need a different UX, maybe you have to click it?

Can we also explore perhaps a side by side mode, or some way we can see more of the params, even in the most trivial manifests I'm always having to scroll.

Screenshot 2024-10-23 at 13 29 08
@zanete zanete moved this to Ready in IF Oct 23, 2024
@zanete zanete added this to the IF 1.0 milestone Oct 23, 2024
@zanete zanete added this to D: Web Oct 23, 2024
@zanete zanete moved this to Todo in D: Web Oct 23, 2024
@osamajandali
Copy link

osamajandali commented Nov 5, 2024

@jawache Here’s a quick prototype of the new flyout with improved UX in side-by-side mode. Attaching a video since it’s still just a dummy prototype and not yet integrated into the visualizer, which should help us iterate faster.

cc: @zanete

new-flyout.mov

@zanete zanete moved this from In Progress to In Review in D: Web Nov 5, 2024
@zanete zanete moved this from Ready to Pending Review in IF Nov 6, 2024
@jawache
Copy link
Contributor Author

jawache commented Nov 6, 2024

Thanks @osamajandali few initial thoughts. I like how it's making the data more useful and helping us see connections. It's using up a lot of space and the manifests we are creating now have many parameters, so I think maybe we need to compress the UI a lot more? Or even turn them into tables rather than buttons?

Also, given the above and how hard it would be to even browse this data on mobile, I'm thinking we perhaps make the flyout "desktop only" and not have to worry about reflowing and mobile UX (I don't see anyone doing any complex analysis on mobile, just looking at the graph and the numbers perhaps!🤷‍♂️) what do you think?

I also just realised that with the pie chart version, since there isn't a time series with cells, there isn't anything to click to bring out the flyout. To fix this I'm thinking now of a version where the flyout has the time series of data and clicking the total cell brings up the flyout, since the flyout will be just for a single component. Will draw up something real quick to share.

@zanete zanete moved this from In Review to In Progress in D: Web Nov 14, 2024
@zanete zanete moved this from Pending Review to In Progress in IF Nov 14, 2024
@jawache
Copy link
Contributor Author

jawache commented Nov 19, 2024

Aaand @osamajandali, @zanete and @jmcook1186 here is a video where I propose an alternative flyout function that works better with the pie chart version. Key features:

  • The flyout is for an entire component, not just one cell, but the whole timeseries. That's the only way it works with the pie chart version which doesn't show cells of timeseries.
  • Merging defaults, inputs and outputs into one table of "parameters".
  • I proposed using arrows but actually the UX above where you click on something and it strips out all the other parameters/models to focus down on just whats relevant also works.

NOTE: You can't assume the flow will be inputs -> model -> outputs, some models will take an output -> model -> another-output, so the exact UI you proposed in the above UX won't work, but maybe if we used arrows or some color coding with one list of "parameters" and one list of "pipeline" models?

@jmcook1186 - i'm err not sure how this works with our two pipelines, observe and compute - perhaps we need to show two pipelines, or maybe merge it all into one, or maybe just compute? Unsure...

https://drive.google.com/file/d/1KQ0tuSvccu9W6A6PTp6KeLAEy3_hRJvZ/view?usp=sharing

@zanete
Copy link

zanete commented Nov 27, 2024

status update: the must haves will be done by end of the week, also hoping for the nice-to-haves in there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Status: In Progress
Development

No branches or pull requests

3 participants