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

Graph UI needs zoom controls #17874

Closed
Tracked by #179668
elasticmachine opened this issue Feb 10, 2017 · 9 comments
Closed
Tracked by #179668

Graph UI needs zoom controls #17874

elasticmachine opened this issue Feb 10, 2017 · 9 comments
Labels
Feature:Graph Graph application feature impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@elasticmachine
Copy link
Contributor

Original comment by @peterskim12:

Maybe I'm a weirdo but I disable zoom functions (along with pretty much all other functions) on my MacBook Pro trackpad to prevent inadvertent actions. I can't find a way to zoom in and out in the Graph UI if I don't use the trackpad.

There should be controls in the Graph UI to zoom in and out similar to Google Maps:

!LINK REDACTED

@elasticmachine
Copy link
Contributor Author

Original comment by @markharwood:

The current trackpad behaviour (at least on my mac) is that a 2 finger swipe up or down will zoom. This is consistent with Google maps behaviour.

I disable zoom functions
I see there is an OSX System preference that disables the zoom action but that is tied to the pinch action not the 2 finger vertical swipe?

Presumably you have 2 finger vertical swipes enabled in order to scroll pages?

The disadvantage with using buttons for zoom is that action is normally targeted on the screen centre and a mouse/trackpad based zoom is more naturally associated with centring on the current location of the mouse. If we add the buttons people may fall into the trap of using them and then they have to pan a lot more to arrive at their area of interest. Maybe some help text describing the current mouse/trackpad targeted zoom behaviour is required?

@elasticmachine
Copy link
Contributor Author

Original comment by @peterskim12:

I've disabled all trackpad gestures intentionally, except for the 2 finger scroll.
Also anyone using a traditional mouse or trackball have no way to zoom in and out.

@elasticmachine
Copy link
Contributor Author

Original comment by @markharwood:

I've disabled all trackpad gestures intentionally, except for the 2 finger scroll.

2 finger scroll is the zoom gesture.

Also anyone using a traditional mouse or trackball have no way to zoom in and out.

Granted, anyone without a mouse scroller wheel will be stuck so buttons make sense.

@elasticmachine
Copy link
Contributor Author

Original comment by @peterskim12:

Ok cool, I didn't realize the 2 finger scroll does zoom also. I assumed the "Pinch with two fingers" was required to zoom.

Yeah I think it's still important to make sure this UI is usable by people who have a traditional 2 button mouse or other input devices that doesn't have a scroll function. At home, I use a Windows 10 Desktop with a Logitech trackball; there's a way to configure it to do scroll but I don't have it set up that way.

@elasticmachine
Copy link
Contributor Author

Original comment by @markharwood:

I hoped this would be a quick fix I could turn around but isn't as easy as I thought.
Zoom-scale and pan position is held as state inside d3 somewhere and updated with mouse scroll actions and passed out on event objects. If we have custom buttons outside of d3 that change zoom and pan position on the SVG DOM elements I need to find a way to also reset the d3-managed scale and pan variables otherwise mouse actions and button actions are incrementing two different notions of the current zoom and pan levels. Things leap around unexpectedly flipping between mouse and buttons with my first simplistic attempt at this.
Also, the pointer-less style of zooming using buttons cannot just increment and decrement scale - it must also adjust pan appropriately each time to remain centred. Need to figure out that logic too.

So parking this one currently due to the priorities of other work.

@elasticmachine
Copy link
Contributor Author

Original comment by @peterskim12:

Cool thanks for making the effort @markharwood.

@elasticmachine elasticmachine added the Feature:Graph Graph application feature label Apr 24, 2018
@timroes timroes added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label Aug 8, 2018
@timroes timroes added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Feature:Visualizations Generic visualization features (in case no more specific feature label is available) labels Sep 16, 2018
@timroes timroes added Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Sep 3, 2021
@elasticmachine
Copy link
Contributor Author

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@stratoula stratoula added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. labels Nov 4, 2022
@elasticmachine
Copy link
Contributor Author

Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations)

@stratoula stratoula added the impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. label Jun 2, 2023
@timductive
Copy link
Member

Closing this because it's not planned to be resolved in the foreseeable future. It will be tracked in our Icebox and will be re-opened if our priorities change. Feel free to re-open if you think it should be melted sooner.

@timductive timductive closed this as not planned Won't fix, can't repro, duplicate, stale Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Graph Graph application feature impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

No branches or pull requests

4 participants