Skip to content

Conversation

@matus-tomlein
Copy link
Contributor

@matus-tomlein matus-tomlein commented Jan 10, 2024

Issue #1280

This reopens the PR #1279 from @jethron that fixes the issue where clicks on buttons with child elements are not recognized.

@bundlemon
Copy link

bundlemon bot commented Jan 10, 2024

BundleMon

Files added (6)
Status Path Size Limits
libraries/browser-tracker-core/dist/index.mod
ule.js
+25.67KB 26KB / +10%
trackers/javascript-tracker/dist/sp.js
+24.76KB 25KB / +10%
trackers/javascript-tracker/dist/sp.lite.js
+15.08KB 15.5KB / +10%
trackers/browser-tracker/dist/index.umd.min.j
s
+14.94KB 15KB / +10%
libraries/tracker-core/dist/index.module.js
+13.36KB 15KB / +10%
trackers/browser-tracker/dist/index.module.js
+3.51KB 5KB / +10%

Total files change +97.32KB 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history

Copy link
Contributor

@greg-el greg-el left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@jethron jethron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @matus-tomlein!

One day I'll take the time to get node@16 working so the tests work locally. :P

@matus-tomlein matus-tomlein changed the base branch from master to release/3.20.0 January 12, 2024 10:48
@matus-tomlein matus-tomlein merged commit e8913f0 into release/3.20.0 Jan 12, 2024
@matus-tomlein matus-tomlein deleted the issue/button_click_with_children branch January 12, 2024 10:49
matus-tomlein added a commit that referenced this pull request Jan 15, 2024
PR #1281 --------- Co-authored-by: Jethro Nederhof <jethro.nederhof@snowplowanalytics.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants