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

Label merging should be solely text-based #4798

Open
3 tasks done
aplavin opened this issue Feb 16, 2025 · 1 comment
Open
3 tasks done

Label merging should be solely text-based #4798

aplavin opened this issue Feb 16, 2025 · 1 comment
Labels

Comments

@aplavin
Copy link
Contributor

aplavin commented Feb 16, 2025

Labels with the same text do not get merged:

scatter(rand(100), label="Abc" => (;markersize=20))
lines!(rand(100), label="Abc")
axislegend(merge=true)

To get merged, the same label arguments need to be specified, even if they don't make any sense for the plot type:

Image

I guess merge=true is supposed to merge labels based on their text, not on their attributes – right?

  • what version of Makie are you running? (]st -m Makie) v0.21
  • can you reproduce the bug with a fresh environment ? (]activate --temp; add Makie)
  • What platform + GPU are you on? any
@aplavin aplavin added the bug label Feb 16, 2025
@jkrumbiegel
Copy link
Member

Yep that's a bug, should add a test case for this scenario

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants