-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove <noscript> from default JS tracking code, new checkbox to add <noscript> + remove<p> tag in <noscript> #9770
Comments
Good point. However, I presume it is there for valid HTML structure. Eg If the Are there any opinions if without |
Actually, since #9341 we recommend to put the tracking code into |
maybe we should
|
Renamed issue to increase scope. |
A way to solve that particular issue is to set the style of the p tag as |
You are both wrong ;) As @braiam points out, However, an So, what to do? My proposed solutionIf you want you want the (X)HTML to be valid (yes, please!) and you want to use an If you do that, then you might as well follow the advice of @hpvd here and @MattiSG here, and put the entire code block just before the |
fyi @HLFH the noscript is removed from Matomo now by default so you can remove it from your tracking code as well. it should fix the issue. |
@mattab Hi Mattab, this issue was the But finally or sadly, and regardless of this small issue, I had trouble to have any working tracking with the last version 4.0.0-b4. So I migrated to GA 4. |
Having img without p works just as fine. the P tag could cause unwanted space in the web page that executes the noscript block.
The text was updated successfully, but these errors were encountered: