Great, I’ll pick up what I did, it should be quick.
@pierre-gilles
In this kind of case, should I open a new PR (actually I think there’s not much choice because I wouldn’t be able to work from his branch?)
In itself the zoomed display on the dashboard is a relatively simple PR without implementing the whole history and the date-to-date feature!! Maybe it’s worth making a small PR about that, @pierre-gilles, no? And we could then graft the ‹ Graph improvements › topic onto it later?
Their thing is pretty hacky to properly intervene on all my searches; despite an option yaxis.labels.style.cssClass (Options (Reference) > yaxis – ApexCharts.js) you have to intervene directly on the SVG. So I broke the ‹ device › part of the ‹ feature › part to force a line break.
I wanted to reduce the font when there are more than two binaries in the chart, what do you think? In the image, the names on the graph on the right look all cramped, don’t they?
I think the line break looks good, I’m not bothered by the graph on the right personally.
Anyway, it’s impossible to fit a device name + a feature in full under any circumstances, in my opinion!
I can reproduce it… But what I don’t understand is that it doesn’t have a direct link with the PR (pull request). It’s the request, according to the console.log I put just after, that takes a long time for the binary only when there aren’t enough values (< 3 or 5 values) … yet we didn’t touch anything on that side ..
So, it was a rendering bug — the chart generation was spinning, I guess because it can’t build the rangeBar chart when the data are below the base min/max scale (which must have been 5 min). Once it had more than 3-4 minutes of data, it was unable to render the dates in the correct format and didn’t display the data despite the curve appearing. And this persisted up to 10 minutes of data.
From all my tests (the time I spent on it ^^) adding the following to the options: