CLS: Add as a tag

jarstelfox · jarstelfox · commit e67b8bdcd5b7 · 2021-06-23T11:23:12.000-07:00
See: https://github.com/getsentry/sentry-javascript/pull/3734\#discussion_r657311310
diff --git a/packages/tracing/src/browser/metrics.ts b/packages/tracing/src/browser/metrics.ts
@@ -211,9 +211,7 @@ export class MetricsInstrumentation {
 
       if (this._clsEntry) {
         logger.log('[Measurements] Adding CLS Data');
-        transaction.setData('measurements.cls', {
-          sources: this._clsEntry.sources.map(source => htmlTreeAsString(source.node)),
-        });
+        this._clsEntry.sources.map((source, index) => transaction.setTag(`cls.source.${index + 1}`, htmlTreeAsString(source.node)));
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -211,9 +211,7 @@ export class MetricsInstrumentation {`
`211`	`211`
`212`	`212`	`if (this._clsEntry) {`
`213`	`213`	`logger.log('[Measurements] Adding CLS Data');`
`214`		`- transaction.setData('measurements.cls', {`
`215`		`- sources: this._clsEntry.sources.map(source => htmlTreeAsString(source.node)),`
`216`		`- });`
	`214`	+ this._clsEntry.sources.map((source, index) => transaction.setTag(`cls.source.${index + 1}`, htmlTreeAsString(source.node)));
`217`	`215`	`}`
`218`	`216`	`}`
`219`	`217`	`}`