-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Dive dashboard launch #5946
Comments
Agree that from a UX perspective, ordering the focused item is a priority. I suppose we can build "wrapping" rules into the layout logic such that it will cause other tiles to move down to the next line — will look deeper into the API here. For highlighting a focus area, the Chartjs crosshair plugin (which we already use) supports dragging. See example. Though, as we talked about at the offsite, I think we should allow the user to select a region, but not automatically zoom into it, as that loses context for the data. |
Also, this minor issue can be fixed in minutes — will get around to it soon: #5917 (merged) |
Sounds great - add a request in the art request board with a deadline so I can factor this into my schedule |
Taking a step back here, I want to make sure we fully understand all the context so we can work on it in the best possible way. CC @clarkus
|
@paolodamico My 2¢ on the following:
Broadly: lack of context.
I am not sure. At this point the "need" is admittedly somewhat theoretical, and that could be why we haven't felt more urgency here. |
That's helpful @samwinslow, thanks! So my proposal would be, let's consider this in the context of Collaboration, both in terms if we want to solve the problem and if this is the best way to do it. If so, we can pick up anything outstanding and ship, and if not we can remove it. Thoughts? CC @alexkim205 |
In addition to the CEO the lack of context problem also exists between junior/senior engineers/data analysts/support heros, e.g. trying to understand what's happening if some graph looks odd. Agree for grouping with collaboration. |
Thanks for the context here. I think there's still a gap in understanding which dimensions we're diving into for a specific insight that needs to be explored here. For example (I might not be understanding how to use this feature exactly) I'm not sure what's happening when I set the dive dashboard of dashboard item, dive from insight to dashboard, and dive back. A simple explanation or copy could clarify what a "dive" actually is. I think this falls under the general "understanding an insight better" collaboration milestone but should be deprioritized until we figure out how to best expose this feature in a usable way. |
Sounds like a plan @alexkim205! I do think we may benefit of rethinking the problem from scratch, just cause it's taking us so long to realize how it works and all the value from the feature. We should have a clear idea of what collaboration could look like for the next few sprints, and decide whether we want to move forward with this or not. |
Yes yes, it's behind a feature flag, but while working on insights, I've had to strip out, refactor or fix bugs coming from code related to dive dashboards on at least 3 different occasions. That's a large blast radius for a hackathon project that is not even used. Let's do better next time. |
We worked on this during the offsite, but there are a couple of things that are still needed to productionize it:
cc @samwinslow @jamesefhawkins anything else I'm missing?
The text was updated successfully, but these errors were encountered: