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

Waffle mark tooltips are always on axis line #2129

Open
ericvdtoorn opened this issue Aug 8, 2024 · 1 comment · May be fixed by #2132
Open

Waffle mark tooltips are always on axis line #2129

ericvdtoorn opened this issue Aug 8, 2024 · 1 comment · May be fixed by #2132
Labels
bug Something isn’t working

Comments

@ericvdtoorn
Copy link

ericvdtoorn commented Aug 8, 2024

image

Ideally, the tooltip should show up if and only if you're hovering on the 2seater group

Basic Repro

// mpg can be downloaded from https://gist.github.com/omarish/5687264/raw/7e5c814ce6ef33e25d5259c1fe79463c190800d9/mpg.csv
const mpg = FileAttachment("data/mpg.csv").csv({typed:true});
const sql = await DuckDBClient.sql({mpg: mpg}) 

Tr

select class, round(100 * count(*) / (select count(*) from mpg) ) as n from mpg group by class order by count(*);
Plot.plot({
    axis: null,
    title: "Waffle Chart",
    subtitle: "'Class of vehicles",
    marks: [
        Plot.waffleX(perc, {
                    x: "n",
                    fill: "class",
                    multiple: 10,
                    tip: true
                    })
    ],
    color: {legend: true},
    height: 400
@ericvdtoorn ericvdtoorn added the bug Something isn’t working label Aug 8, 2024
@mbostock
Copy link
Member

mbostock commented Aug 8, 2024

Yes, we’re aware it works like this. Not sure it’s fixable given how the underlying coordinate system is defined (band scale), but we might be able to do something.

@Fil Fil linked a pull request Aug 12, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn’t working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants