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

Force re-layout not working #2596

Closed
abuehrle opened this issue Jun 13, 2017 · 4 comments
Closed

Force re-layout not working #2596

abuehrle opened this issue Jun 13, 2017 · 4 comments

Comments

@abuehrle
Copy link
Contributor

Version: weaveworks/scope:release-1.5-1667984
Docker for Mac
Sockshop
Standalone Scope

Please see:

http://recordit.co/cZshm6AwXh

@abuehrle abuehrle mentioned this issue Jun 13, 2017
36 tasks
@fbarl
Copy link
Contributor

fbarl commented Jun 13, 2017

@abuehrle What was your expected behaviour? Take a look at my comment here: #2564 (comment)

I think people have very different ideas what the refresh button should be doing, see the issue #2377 and my "fix" #2407. :)

@abuehrle
Copy link
Contributor Author

I expected the containers to centre themselves within that view so that you can see all of the containers including the ones that have fallen out of the main view. When you click the force layout, it appears as if nothing happens. So the zoom correction was removed? What should I be looking for when I force re-layout?

@fbarl
Copy link
Contributor

fbarl commented Jun 13, 2017

Yeah, most people seemed very opinionated about this :)

The idea with the refresh button is that it would force relayouting on the graph, meaning the nodes would rearrange if they were not positioned "optimaly", while the zoom state is currently left unchanged by design.

@abuehrle
Copy link
Contributor Author

Ok... I will leave that one.

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

No branches or pull requests

2 participants