8000 Adding the option to have no legend by federicomarini · Pull Request #689 · iSEE/iSEE · GitHub
[go: up one dir, main page]

Skip to content

Adding the option to have no legend#689

Merged
kevinrue merged 4 commits intodevelfrom
addoption_nolegend
Mar 6, 2025
Merged

Adding the option to have no legend#689
kevinrue merged 4 commits intodevelfrom
addoption_nolegend

Conversation

@federicomarini
Copy link
Contributor

Adding the option to have NO legend on the DotPlot panels

  • adding the constants
  • change the documentation to reflect the additional option

- adding the constants
- change the documentation to reflect the additional option
@federicomarini
Copy link
Contributor Author

Just for the formal aspects - if someone among @csoneson @kevinrue @LTLA want to check this out, feel free to do so. Otherwise it is ready to be merged (all checks passing so far)

@kevinrue
Copy link
Member
kevinrue commented Feb 27, 2025

babysitting duty tonight - i was almost looking forward to coding that one myself, but lack time and energy and appreciate that you did it ;)

@lee-t
Copy link
lee-t commented Mar 5, 2025

Friendly reminder for reviewers to check out the PR. Would be nice to have this on devel this week.

@kevinrue
Copy link
Member
kevinrue commented Mar 6, 2025

On it today. Just gotta bring my renv up to date 😬

@kevinrue
Copy link
Member
kevinrue commented Mar 6, 2025

Indeed it works, but I noticed that in the code tracker, the following line then becomes effectively redundant

guides(colour = guide_legend(override.aes = list(size=1)), fill = guide_legend(override.aes = list(size=1))) +

I guess that can be adjusted later, as long as the expected behaviour is implemented. I'm just too short on time right now to do that carefully.

@kevinrue kevinrue merged commit 55c1d18 into devel Mar 6, 2025
3 checks passed
@kevinrue kevinrue deleted the addoption_nolegend branch March 6, 2025 10:29
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

0