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.

Sign up for GitHub

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

Jump to bottom

projection.inset? #162

Closed
mbostock opened this issue Mar 26, 2019 · 2 comments
Closed

projection.inset? #162

mbostock opened this issue Mar 26, 2019 · 2 comments
Labels
idea

Comments

@mbostock
Copy link
Member

I often find myself wanting to projection.fitWidth (related #161), but then reserving 1px or 10px of margin around the fit object. There’s no easy way to do this currently, unlike with projection.fitExtent.
@Fil
Copy link
Member

Fil commented Sep 18, 2019

inset can be passed before ( projection.inset(10).fitWidth(…) ) but could alternatively be an option of fitWidth:
projection.fitWidth(width, object, {inset: 10})

In terms of usability I'd love to have {type:Sphere} as the default, but this idea would seem to clash with that one.

@Fil Fil mentioned this issue Jun 17, 2020
Open
@Fil Fil added the idea label Jul 10, 2020
@Fil Fil mentioned this issue Apr 26, 2021
Open
@Fil
Copy link
Member

Fil commented Dec 16, 2022 •
edited

closing due to inactivity (note that Plot has this shorthand)

@Fil Fil closed this as completed Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
idea
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Fil @mbostock