Skip to content

change geom_vline and geom_hline inherit.aes parameter default to NULL#6836

Open
EvaMaeRey wants to merge 2 commits intotidyverse:mainfrom
EvaMaeRey:patch-2
Open

change geom_vline and geom_hline inherit.aes parameter default to NULL#6836
EvaMaeRey wants to merge 2 commits intotidyverse:mainfrom
EvaMaeRey:patch-2

Conversation

@EvaMaeRey
Copy link
Copy Markdown
Contributor

No description provided.

inherit.aes = FALSE) {
inherit.aes = NULL) {

# if NJLL default is based on if xintercept is missing
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# if NJLL default is based on if xintercept is missing
# if NULL, default is based on whether xintercept is missing

(And same below).

@arcresu
Copy link
Copy Markdown
Contributor

arcresu commented Apr 13, 2026

I wonder whether the docs should also be updated to explain how the behaviour changes based on xintercept's presence (see also #6782).

@teunbrand
Copy link
Copy Markdown
Collaborator

Gina does this PR aim to fix #6831?

@arcresu
Copy link
Copy Markdown
Contributor

arcresu commented Apr 14, 2026

I think this is for #6833, which is separate. If this is merged, the mapping would still be thrown out when xintercept is specified, which is what #6831 is about. I updated the title there to hopefully be clearer.

@EvaMaeRey
Copy link
Copy Markdown
Contributor Author

That's right, @arcresu. This is about #6833 ...

@EvaMaeRey EvaMaeRey changed the title change geom_abline and friends inherit.aes parameter default to NULL change geom_vline and geom_hline inherit.aes parameter default to NULL Apr 14, 2026
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

Successfully merging this pull request may close these issues.

3 participants