Replace inline onclick with data attribute for click tracking

This commit is contained in:
rulingcom 2026-07-31 19:06:21 +08:00
parent 6422dc14ae
commit bb6cd30fb4
9 changed files with 11 additions and 11 deletions

View File

@ -35,7 +35,7 @@ class WebResourcesController < ApplicationController
"category" => link.category.title,
"link_to_show" => link.url,
"target" => target,
"onclick" => "if(navigator.sendBeacon){navigator.sendBeacon('/web_resources/track_click/#{link.id}');}"
"track_click_url" => "/web_resources/track_click/#{link.id}"
}
end
{
@ -82,7 +82,7 @@ class WebResourcesController < ApplicationController
"statuses" => statuses,
"link_to_show" => link.url,
"target" => target,
"onclick" => "if(navigator.sendBeacon){navigator.sendBeacon('/web_resources/track_click/#{link.id}');}"
"track_click_url" => "/web_resources/track_click/#{link.id}"
}
end
{

View File

@ -7,7 +7,7 @@
<div class="link-img-wrap{{display_image}}">
<img src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}">
</div>
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">{{title}}</a>
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">{{title}}</a>
<span data-list="statuses" data-level="1">
<span class="label status {{status-class}}">{{status}}</span>
</span>

View File

@ -7,7 +7,7 @@
<div class="link-img-wrap{{display_image}}">
<img src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}">
</div>
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">{{title}}</a>
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">{{title}}</a>
<span data-list="statuses" data-level="1">
<span class="label status {{status-class}}">{{status}}</span>
</span>

View File

@ -12,7 +12,7 @@
data-cycle-pause-on-hover="true" data-cycle-log="false" data-cycle-carousel-fluid="true">
<li class="widget-link__item">
<div class="liWrapper">
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">
<img class="widget-link__img" src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}">
</a>
</div>

View File

@ -12,7 +12,7 @@
data-cycle-pause-on-hover="true" data-cycle-log="false" data-cycle-carousel-fluid="true">
<li class="widget-link__item">
<div class="liWrapper">
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">
<img class="widget-link__img" src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}"><br />
{{title}}
</a>

View File

@ -12,7 +12,7 @@
data-cycle-pause-on-hover="true" data-cycle-log="false" data-cycle-carousel-fluid="true">
<li class="widget-link__item">
<div class="liWrapper">
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">
<a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">
<img class="widget-link__img" src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}"><br />
</a>
</div>

View File

@ -14,9 +14,9 @@
<div class="liWrapper">
<img class="widget-link__img" src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}">
</div>
<h4 class="widget-content-title"><a href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">{{title}}</a></h4>
<h4 class="widget-content-title"><a href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">{{title}}</a></h4>
<br>
<div class="widget-content-context"><a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">{{context}}</a></div>
<div class="widget-content-context"><a class="widget-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">{{context}}</a></div>
</li>
</div>
<ul class="button-mid">

View File

@ -4,7 +4,7 @@
</h1>
<ul class="list-unstyled" data-list="web_link" data-level="0">
<li class="index-content">
<a class="index-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">
<a class="index-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">
<h4>
<div class="link-img-wrap{{display_image}}">
<img src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}">

View File

@ -4,7 +4,7 @@
</h1>
<ul class="list-unstyled" data-list="web_link" data-level="0">
<li class="index-content col-md-4 col-sm-4">
<a class="index-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" onclick="{{onclick}}">
<a class="index-content-title" href="{{link_to_show}}" target="{{target}}" title="{{title_text}}" data-track-click-url="{{track_click_url}}">
<div class="link-img-wrap{{display_image}}">
<img src="{{img_src}}" alt="{{img_description}}" title="{{img_description}}">
</div>