prfalken/leonardo

Graphs not shown in multiple search

MasterofJOKers opened this issue · 0 comments

From what I gathered so far, the url for the images is really set via JavaScript and there needs to be a div around, that has the class needed_width. There is no such div in templates/multiple.html. Thus https://github.com/PrFalken/leonardo/blob/master/leonardo/static/js/leonardo.js#L178 replaces the width with null and graphite throws an error: ValueError: invalid literal for int() with base 10: 'null'

Imho there is no way to get the correct width of the images in multiple view, because there might be any number of graphs that have to fit in one line so multiple dashboards can be compared. So maybe a default should be used if needed_width is not set.