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

Allow bounds-expansion to set a different size for each side #2499

Merged

Conversation

@josejulio
Copy link
Contributor

commented Aug 13, 2019

This is done by accepting 1, 2 or 4 params.

Related to #2497

josejulio added 3 commits Aug 13, 2019
- expandBoundingBoxSides default padding is now [0]
@josejulio josejulio force-pushed the josejulio:make-bounds-expansion-flexible branch from f80181d to 0e669cf Aug 13, 2019
@maxkfranz maxkfranz self-requested a review Aug 15, 2019
Copy link
Member

left a comment

I've suggested some minor changes to the documentation.

documentation/md/style.md Outdated Show resolved Hide resolved
Updates wording in docs

Co-Authored-By: Max Franz <maxkfranz@gmail.com>
@josejulio

This comment has been minimized.

Copy link
Contributor Author

commented Aug 16, 2019

I've suggested some minor changes to the documentation.

Done, thanks!

@maxkfranz maxkfranz added this to the 3.10.0 milestone Aug 19, 2019
@maxkfranz maxkfranz merged commit c811a45 into cytoscape:unstable Aug 19, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@josejulio josejulio deleted the josejulio:make-bounds-expansion-flexible branch Aug 19, 2019
@maxkfranz maxkfranz referenced this pull request Sep 4, 2019
6 of 6 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.