The chart was not fitting to the height of the screen in Firefox. Other than introducing an inconsistency between browsers, this made its height dependent on the height of the side navigation.
Demo (Firefox screenshots)
Sidebar full height (scrolls)
Sidebar partially collapsed
Notes
Unrelated to this issue, the angular component <ccl-sidebar> should have its class changed from class="sidebar-content scrollable" to class="sidebar". This will need to be addressed in a separate PR and shouldn't affect UI.
It doesn't seem critical, but could be worthwhile to consider adding a min-height to the chart, as it gets pretty small at certain screen heights.
Overview
The chart was not fitting to the height of the screen in Firefox. Other than introducing an inconsistency between browsers, this made its height dependent on the height of the side navigation.
Demo (Firefox screenshots)
Sidebar full height (scrolls)
Sidebar partially collapsed
Notes
<ccl-sidebar>
should have its class changed fromclass="sidebar-content scrollable"
toclass="sidebar"
. This will need to be addressed in a separate PR and shouldn't affect UI.Closes|Fixes azavea/climate-change-components#13