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

Unified Diff: options.html

Issue 29334038: Issue 2802/2358 - Dynamically generate tooltips in options page (Closed)
Patch Set: Created Jan. 19, 2016, 6:24 p.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
« no previous file with comments | « locale/en-US/options.json ('k') | options.js » ('j') | skin/options.css » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.html
===================================================================
--- a/options.html
+++ b/options.html
@@ -83,17 +83,9 @@
<div id="blocking-languages">
<div class="option-name">
<span class="i18n_options_language_title"></span>
- <div class="tooltip">
- <span class="i18n_options_readMore"></span>
- <div role="tooltip">
- <img src="skin/tooltips/block.png" alt="" />
- <p class="i18n_options_language_title_tooltip"></p>
- <div class="notes">
- <p class="i18n_options_language_title_tooltip_notes_1"></p>
- <p class="i18n_options_language_title_tooltip_notes_2"></p>
- </div>
- </div>
- </div>
+ <span class="i18n_options_readMore read-more"
+ data-tooltip="options_language_title_tooltip"
+ data-tooltip-image="skin/tooltips/block.png"></span>
</div>
<ul id="blocking-languages-table" class="table list">
<template>
@@ -110,20 +102,22 @@
</div>
<div id="further-blocking">
<div class="option-name">
- <strong class="i18n_options_furtherBlocking_title"></strong>
- <div class="tooltip">
- <span class="i18n_options_readMore"></span>
- <div role="tooltip" class="flip-vertical">
- <img src="skin/tooltips/more.png" alt="" />
- <p class="i18n_options_furtherBlocking_title_tooltip"></p>
- </div>
- </div>
+ <strong class="i18n_options_more_title"></strong>
+ <span class="i18n_options_readMore read-more"
+ data-tooltip="options_more_title_tooltip"
+ data-tooltip-image="skin/tooltips/more.png"
+ data-tooltip-flip="horizontal"></span>
</div>
<div id="custom-wrapper">
<ul id="recommend-list-table" class="table list">
<template>
<input type="checkbox" class="control" />
- <span class="display"></span>
+ <div class="fill">
+ <span class="display"
+ data-tooltip="options_more_%value%_tooltip"
+ data-tooltip-value=""
+ data-tooltip-native></span>
+ </div>
<span class="popular"></span>
</template>
</ul>
@@ -137,7 +131,7 @@
<div class="controls">
<button data-action="open-dialog" data-dialog="custom">
<span class="icon icon-add"></span>
- <span class="i18n_options_furtherBlocking_add"><span>
+ <span class="i18n_options_more_add"><span>
</button>
</div>
</div>
@@ -149,15 +143,9 @@
<div id="acceptable-ads">
<div class="option-name">
<strong class="i18n_options_acceptableAds_title"></strong>
- <div class="tooltip">
- <span class="i18n_options_readMore"></span>
- <div role="tooltip">
- <img src="skin/tooltips/acceptable-ads.png" alt="" />
- <p class="i18n_options_acceptableAds_title_tooltip_1"></p>
- <p class="i18n_options_acceptableAds_title_tooltip_2"></p>
- <p class="i18n_options_acceptableAds_title_tooltip_3"></p>
- </div>
- </div>
+ <span class="i18n_options_readMore read-more"
+ data-tooltip="options_acceptableAds_title_tooltip"
+ data-tooltip-image="skin/tooltips/acceptable-ads.png"></span>
</div>
<ul id="acceptableads-table" class="table list">
<template>
@@ -169,17 +157,10 @@
<div id="whitelisting">
<div class="option-name">
<strong class="i18n_options_whitelisted_title"></strong>
- <div class="tooltip">
- <span class="i18n_options_readMore"></span>
- <div role="tooltip" class="flip-vertical">
- <img src="skin/tooltips/whitelisted.png" alt="" />
- <p class="i18n_options_whitelisted_title_tooltip_1"></p>
- <p class="i18n_options_whitelisted_title_tooltip_2"></p>
- <div class="notes">
- <p class="i18n_options_whitelisted_title_tooltip_notes"></p>
- </div>
- </div>
- </div>
+ <span class="i18n_options_readMore read-more"
+ data-tooltip="options_whitelisted_title_tooltip"
+ data-tooltip-image="skin/tooltips/whitelisted.png"
+ data-tooltip-flip="horizontal"></span>
</div>
<ul id="whitelisting-table" class="table list">
<template>
@@ -213,14 +194,14 @@
<!-- Advanced tab content -->
<div id="content-advanced">
<div>
- <h1><span class="i18n_options_tweaks_title"></span><a class="i18n_options_readMore tooltip" href="#"></a></h1>
+ <h1><span class="i18n_options_tweaks_title"></span><a class="i18n_options_readMore read-more tooltip" href="#"></a></h1>
<ul class="table" style="width: auto;">
<li>
<input type="checkbox" id="easylist"/><span id="block-element-explanation" class="i18n_options_tweaks_blockElement"></span></li>
</ul>
</div>
<div>
- <h1><span class="i18n_options_blockingList_title"></span><a class="i18n_options_readMore tooltip" href="#" target="_blank"></a></h1>
+ <h1><span class="i18n_options_blockingList_title"></span><a class="i18n_options_readMore read-more tooltip" href="#" target="_blank"></a></h1>
<ul id="blocking-list-tabs" class="tabs horizontal">
<li class="i18n_options_tab_overview active" data-show="blocking-list-overview"></li><li class="i18n_options_tab_ownList" data-show="custom-filters"></li>
</ul>
« no previous file with comments | « locale/en-US/options.json ('k') | options.js » ('j') | skin/options.css » ('J')

Powered by Google App Engine
This is Rietveld