Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

firefox rendering bug: individual widget charts clipped at 150px tall #83

Closed
fche opened this Issue Sep 16, 2015 · 11 comments

Comments

Projects
None yet
3 participants
Contributor

fche commented Sep 16, 2015

On firefox 40.0.3 (Fedora21), every widget appears clipped to the top 150 pixels, dropping the bottom axis & labels. Chrome & epiphany on same platform seem fine. Browsing with firebug unfortunately not enough for this newb to diagnose the cause. It appears to be a regression in 1.0.2 since 1.0.1.
vector-1 0 2-format

Contributor

fche commented Sep 16, 2015

This problem persists to git master vector, not just the v1.0.2 release.

Collaborator

hc000 commented Sep 16, 2015

I think there is a difference in the way firefox implements svg vs chrome.
To fix this problem in firefox, just need to set the size with px instead of %

screen shot 2015-09-16 at 10 50 50 am

Contributor

spiermar commented Sep 16, 2015

Interesting. The 100% is coming from the default nvd3 css. There's an inline element style setting the px value, but Firefox is ignoring it. That's why FF is trying to use the % value.
I'm using inline because this is calculated based on the widget config. Have to figure out why FF is ignoring it.
screen shot 2015-09-16 at 1 32 14 pm
screen shot 2015-09-16 at 1 32 49 pm

Contributor

spiermar commented Sep 16, 2015

Setting the style manually works, but for some reason, d3 is not setting it on FF.

This should take care of setting the inline style:

.style({'height': height})

Contributor

spiermar commented Sep 16, 2015

Fixed it, just waiting to get my access back. Was away for so long they forgot about me. :-)

Contributor

spiermar commented Sep 16, 2015

2a6f45c

Can you check @fche ?

Contributor

fche commented Sep 16, 2015

That does it, thanks!

@fche fche closed this Sep 16, 2015

@fche fche reopened this Sep 16, 2015

Contributor

fche commented Sep 16, 2015

Aw man, still seing the phenomenon on -some- of the charts.
Yeah, the "CPU Utilization", "Memory Utilization" and "Page Faults" views.- the bar charts?

Collaborator

hc000 commented Sep 16, 2015

@fche I just made a commit, can you try now?

Contributor

spiermar commented Sep 16, 2015

Oops, missed the stacked charts. Tks @hc000 !

Contributor

spiermar commented Sep 18, 2015

Assuming this is working now. Closing the issue, but let me know if I'm wrong. ;-)

@spiermar spiermar closed this Sep 18, 2015

aeg-aeg pushed a commit to aeg-aeg/pcpfans that referenced this issue Dec 5, 2015

upgrade to vector 1.0.2
... but see also Netflix/vector#83 for
a rendering problem on firefox
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment