Skip to content

Add linewidth option geom_path and geom_point. #1085

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

Closed
wants to merge 1 commit into from
Closed

Add linewidth option geom_path and geom_point. #1085

wants to merge 1 commit into from

Conversation

Symaxion
Copy link
Contributor

This PR adds an aesthetic option to geom_point and geom_path (and its derivatives, such as geom_line) to customize the width of the lines drawn (by passing the linewidth property). This is implemented by simply delegating to the lwd option of the grid library.

@hadley
Copy link
Member

hadley commented Jun 11, 2015

Why do you need this instead of size?

@hadley hadley closed this Jun 11, 2015
@Symaxion
Copy link
Contributor Author

I didn't realize the size option was available for geom_path. What I was specifically looking for was increasing the border thickness of geom_point shapes 21-25, which seems to be a common issue:

I only really added the same option to geom_path as an afterthought. If you want, I can resubmit the patch only for geom_point.

@hadley
Copy link
Member

hadley commented Jun 15, 2015

Ah, got it - yes, I'd love a PR that just modified linewidth for geom_point. Maybe call the aesthetic stroke? Or strokewidth? I think you'll also need to update draw_legend() and it might be worthwhile thinking about what the default scale for stroke should be.

@lock
Copy link

lock bot commented Jan 19, 2019

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/

@lock lock bot locked and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants