search

LukeWoodward / SplitsBrowser

Orienteering results analysis
GNU General Public License v2.0
13 stars 9 forks source link

Chart wraps over to new line after jQuery upgrade #64

Closed LukeWoodward closed 5 years ago

LukeWoodward commented 5 years ago

I upgraded jQuery to version 3.4.1 and then found a display issue with the chart wrapping over to a new line. This means that all you see of the chart is the numbers along the top of the chart at the bottom of the screen. Most of the screen is therefore left empty.

Clearly this is undesirable and should be fixed.

LukeWoodward commented 5 years ago

Fix appears to be to calculate the width of the competitor list 'component' using the container div rather than the list div. Firefox includes the width of the scrollbar in the width of the list div whereas other browsers don't and hence end up making the chart too wide and causing it to wrap.