fix: class string separator

PostHog · Dec 12, 2023 · 337a005 · 337a005
1 parent 0cb5405
commit 337a005
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/__tests__/autocapture.test.ts b/src/__tests__/autocapture.test.ts
@@ -1123,7 +1123,7 @@ describe('Autocapture system', () => {
             const props1 = getCapturedProps(newLib.capture)
 
             expect(props1['$elements_chain']).toBe(
-                'a.test-class,test-class2,test-class3,test-class4,test-class5:nth-child="1"nth-of-type="1"href="http://test.com"attr__href="http://test.com"attr__class="test-class,test-class2,test-class3,test-class4,test-class5";span:nth-child="1"nth-of-type="1"'
+                'a.test-class.test-class2.test-class3.test-class4.test-class5:nth-child="1"nth-of-type="1"href="http://test.com"attr__href="http://test.com"attr__class="test-class test-class2 test-class3 test-class4 test-class5";span:nth-child="1"nth-of-type="1"'
             )
         })
     })

diff --git a/src/autocapture.ts b/src/autocapture.ts
@@ -110,7 +110,7 @@ const autocapture = {
                     // html attributes can _technically_ contain linebreaks,
                     // but we're very intolerant of them in the class string,
                     // so we strip them.
-                    value = splitClassString(value).join()
+                    value = splitClassString(value).join(' ')
                 }
                 props['attr__' + attr.name] = limitText(1024, value)
             }