1 Commits

Author SHA1 Message Date
Sebastian Seedorf
c754316935 fix: replace addEventListener with onmouseleave to prevent listener leak 2026-06-05 12:54:37 +02:00
2 changed files with 9 additions and 7 deletions

View File

@@ -24,13 +24,15 @@ export default function UpsChart({ config, upsRows, timeMode, onEdit, onDelete }
(el, w, h, lRef) => {
if (upsRows.length < 2) return null;
const data = timeMode === 'tick'
? [...upsRows].sort((a, b) => a.game_tick - b.game_tick)
: upsRows;
const xs = data.map((r) =>
const sorted = [...upsRows].sort((a, b) =>
timeMode === 'tick'
? a.game_tick - b.game_tick
: new Date(a.real_time).getTime() - new Date(b.real_time).getTime(),
);
const xs = sorted.map((r) =>
timeMode === 'tick' ? r.game_tick : new Date(r.real_time).getTime() / 1000,
);
const ys = data.map((r) => r.ups);
const ys = sorted.map((r) => r.ups);
const xAxis: uPlot.Axis = {
...AXIS_BASE,

View File

@@ -57,11 +57,11 @@ export function usePlot(
// Defer mouseleave — prevents React hydration events firing before lastIdxRef set
requestAnimationFrame(() => {
plot.over.addEventListener('mouseleave', () => {
plot.over.onmouseleave = () => {
const p = plotRef.current;
if (!p) return;
p.setCursor({ left: idxToPixel(p, lastIdxRef.current), top: -10 });
});
};
});
}
// deps is intentionally dynamic — passed by parent to allow external rebuild triggers