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

[Lens] Selecting a suggested visualization doesn't switch the current visualization title #92624

Closed
Tracked by #184459
markov00 opened this issue Feb 24, 2021 · 6 comments
Labels
enhancement New value added to drive a business result Feature:Lens 🧊 iceboxed impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. papercut Small "burr" in the product that we should fix. Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@markov00
Copy link
Member

Kibana version:
7.12

Original install method (e.g. download page, yum, from source, etc.):
cloud
Describe the bug:
When you select and click a suggested visualization, the current visualization title on the suggestion panel doesn't change to the selected one, creating confusion. A Refresh button appears and will "cleaup" the state, but seems a bit strange that this need a another user interaction to get the title correctly positioned to the current visualization

Steps to reproduce:

  1. create a Lens visualization
  2. click on one suggested visualization
  3. the suggested visualization is focused/clicked by the current visualization title remain displayed on the first visualization on the left.

Expected behavior:
The current visualization title should be placed exactly on the selected visualization, not only on the first visualization on the left.

Screenshots (if relevant):

Screenshot 2021-02-24 at 15 39 50

Errors in browser console (if relevant):
n/a

@markov00 markov00 added bug Fixes for quality problems that affect the customer experience Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Lens labels Feb 24, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@flash1293
Copy link
Contributor

flash1293 commented Feb 24, 2021

That's by design at the moment - but I understand your confusion.

The idea here is on switching to a suggestion, you are "peeking" into this suggestion. As soon as you hit refresh (or start editing something in the chart), it "commits" the suggestion, and it becomes the new "Current visualization".

People were confused about this a bunch of times (especially what "Refresh" means here).

I'm going to relabel as enhancement for now.

cc @MichaelMarcialis

@flash1293 flash1293 added enhancement New value added to drive a business result and removed bug Fixes for quality problems that affect the customer experience labels Feb 24, 2021
@MichaelMarcialis
Copy link
Contributor

Yes, I believe I mentioned experiencing similar confusion regarding this behavior when we were updating the suggestion buttons' focus states. I completely agree with @markov00's take here.

@flash1293, you mention this is done intentionally as a means to preview a suggestion before committing to it. Is this because committing to a suggestion currently updates the available suggestions (thus potentially leaving the user in a position where they cannot go back to their previous visualization state)? If so, I imagine such concerns could be remedied by 1) the inclusion of an undo feature and/or 2) preventing the suggestions from updating until additional modifications have been made the visualization to warrant new suggestions being presented. Thoughts?

@wylieconlon
Copy link
Contributor

@MichaelMarcialis Your intuition is exactly right, we didn't want users to get stuck. To respond to your ideas specifically:

  1. We haven't talked about undo/redo in a while because a generic undo feature is more complicated, here's the issue for it. Maybe we can find a way to reduce the scope.

  2. That is exactly how the suggestions behave today. If you select a suggestion, it loads the state as a preview. If you modify the state then the suggestion gets fully applied and new suggestions are loaded.

@flash1293
Copy link
Contributor

We wanted to user to be able to switch around suggestions quickly - an undo/redo feature would help with this and be a useful feature in itself, but it's not exactly the same because the flow would become "select suggestion - undo - select next suggestion" instead of "select suggestion - select next suggestion".

If we don't think clicking through the suggestions is a thing we want to highlight, then we could remove this feature once we have undo/redo, but I hope we can come up with a better system that's more intuitive.

@stratoula stratoula added the impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. label Feb 15, 2023
@stratoula stratoula added papercut Small "burr" in the product that we should fix. impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. and removed impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. labels May 26, 2023
@markov00
Copy link
Member Author

In order to provide better transparency of priorities, issues that will not be prioritized within the next 24 months are being closed.

Tracking request in Lens general improvements ice box #184459

@markov00 markov00 closed this as not planned Won't fix, can't repro, duplicate, stale May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Lens 🧊 iceboxed impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. papercut Small "burr" in the product that we should fix. Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

No branches or pull requests

6 participants