Skip to content

Firefox: widget height continuously increases#2452

Merged
arikfr merged 1 commit intogetredash:masterfrom
kravets-levko:bug/firefox-widget-height-continuously-increases
Apr 11, 2018
Merged

Firefox: widget height continuously increases#2452
arikfr merged 1 commit intogetredash:masterfrom
kravets-levko:bug/firefox-widget-height-continuously-increases

Conversation

@kravets-levko
Copy link
Copy Markdown
Collaborator

@kravets-levko kravets-levko commented Apr 10, 2018

Firefox, dashboard, auto-height feature: in some cases (loader shown and widget has params, but possibly other cases) widget starts continuously growing. Why: Firefox uses different rounding strategy for element's dimensions (offsetHeight is always integer (rounded up), getBoundingClientRect returns dimensions with a "garbage", i.e. if element has height 100.45px, it will return something like 100.45006791123px, but in fact will use only few fraction digits). Fix: explicitly round values before computing widget height.

…and widget has params, but possibly other cases) widget start continuously growing
@arikfr
Copy link
Copy Markdown
Member

arikfr commented Apr 10, 2018

Did you check this one on Chrome? 🤣

@kravets-levko
Copy link
Copy Markdown
Collaborator Author

Yes, of course 😎

@arikfr arikfr merged commit a8a9110 into getredash:master Apr 11, 2018
@kravets-levko kravets-levko deleted the bug/firefox-widget-height-continuously-increases branch February 11, 2019 16:49
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.

2 participants