Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: pages/filters/popup.tmpl

Issue 29755578: Issue 2148 - Rewrite the testpages test suite (Closed)
Patch Set: Addressed more feedback. Added README. Created June 7, 2018, 10:20 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pages/filters/popup.tmpl
===================================================================
--- a/pages/filters/popup.tmpl
+++ b/pages/filters/popup.tmpl
@@ -1,4 +1,4 @@
-title = $popup - ABP Test Pages
+title = $popup
template = testcase
{% set testcase_moreinfo = [
@@ -16,6 +16,7 @@
<section class="site-panel">
<h2>$popup</h2>
<p>Check that usage of the $popup filter option is working as expected.</p>
+ <p>With the filter displayed below each test case added to ABP (or with the testcase subscription installed and active), the popups triggered by the link/buttons below should be blocked/closed.</p>
</section>
<section class="site-panel">
@@ -24,7 +25,7 @@
<a href="/en/popuptarget" target="_blank">Link popup</a><br><br>
</div>
<h3>Filters</h3>
- ||testpages.adblockplus.org/en/popuptarget^$popup
+ <pre>||testpages.adblockplus.org/en/popuptarget^$popup</pre>
</section>
<section class="site-panel">
@@ -33,5 +34,5 @@
<button onclick="do_popup();">window.open() popup</button>
</div>
<h3>Filters</h3>
- ||testpages.adblockplus.org/en/popuptarget^$popup
+ <pre>||testpages.adblockplus.org/en/popuptarget^$popup</pre>
</section>
« README.md ('K') | « pages/filters/ping.tmpl ('k') | pages/filters/script.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld