From 5d79974f01973d80dceda083560c772c59a8c284 Mon Sep 17 00:00:00 2001 From: Lukas Stracke Date: Fri, 9 Aug 2024 18:14:45 +0200 Subject: [PATCH] remove todo --- packages/browser-utils/src/metrics/cls.ts | 3 --- 1 file changed, 3 deletions(-) diff --git a/packages/browser-utils/src/metrics/cls.ts b/packages/browser-utils/src/metrics/cls.ts index 7227358486f4..aa25a54754a1 100644 --- a/packages/browser-utils/src/metrics/cls.ts +++ b/packages/browser-utils/src/metrics/cls.ts @@ -88,9 +88,6 @@ function sendStandaloneClsSpan(clsValue: number, entry: LayoutShift | undefined, const duration = msToSec(entry?.duration || 0); const routeName = getCurrentScope().getScopeData().transactionName; - // TODO: Is this fine / does it provide any value? Alternatively, we can - // - send the CLS source node as an attribute - // - do nothing at all and ignore the source node const name = entry ? htmlTreeAsString(entry.sources[0]?.node) : 'Layout shift'; const attributes: SpanAttributes = dropUndefinedKeys({