Data Explorer: Scatterplot size ignored in Firefox
dylanross3 opened this issue · comments
Application or Package Used
Data Explorer
Describe the bug
Selecting a size for points in the Scatterplot chart type has no effect in Firefox. Works fine in Chrome.
To Reproduce
Steps to reproduce the behavior:
- Go here to load the "Happiness" example notebook
- Select the Scatterplot chart type
- Select something in the Size dropdown
- See the points do not resize in Firefox
Expected behavior
The points should resize in Firefox. It should look pretty close to how it looks in Chrome.
Screenshot - Firefox (not correct)
Screenshot - Chrome (correct)
Apparently, Firefox hasn't implemented setting r via CSS style: https://stackoverflow.com/questions/51551729/styling-of-svg-circle-doesn%C2%B4t-work-in-firefox-browser-removes-radius-property?noredirect=1&lq=1
This will require a patch release to Semiotic in order to pass the radius as an attribute as well as via style but it's a minor change.
@dylanross3 when you get a chance pull the latest version and let me know if this resolves the problem. I tested it locally and seems to be fine but I just want to make sure.
Reopening. I thought this was fixed but that's only for initial render. When I change which field is selected for size, the diameter of the data points does not update in Firefox. As a workaround, switching to another chart type and back produces the expected visualization.
This issue hasn't had any activity on it in the last 90 days. Unfortunately we don't get around to dealing with every issue that is opened. Instead of leaving issues open we're seeking to be transparent by closing issues that aren't being prioritized. If no other activity happens on this issue in one week, it will be closed.
It's more than likely that just by me posting about this, the maintainers will take a closer look at these long forgotten issues to help evaluate what to do next.
If you would like to see this issue get prioritized over others, there are multiple avenues 🗓:
- Ask how you can help with this issue 👩🏿💻👨🏻💻
- Help solve other issues the team is currently working on 👨🏾💻👩🏼💻
- Donate to nteract so we can support developers to work on these features and bugs more regularly 💰🕐
Thank you!
Transferring to data explorer repo.
Triaging as "needs: testing" to determine if this is still an issue.