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

Side by Side Diff: chrome/content/ui/firstRun.html

Issue 11533106: Prepared adblockplus for Safari (Closed)
Patch Set: Created Sept. 19, 2013, 2:27 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/content/ui/firstRun.js » ('j') | chrome/content/ui/firstRun.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <!-- 2 <!--
3 - This file is part of Adblock Plus <http://adblockplus.org/>, 3 - This file is part of Adblock Plus <http://adblockplus.org/>,
4 - Copyright (C) 2006-2013 Eyeo GmbH 4 - Copyright (C) 2006-2013 Eyeo GmbH
5 - 5 -
6 - Adblock Plus is free software: you can redistribute it and/or modify 6 - Adblock Plus is free software: you can redistribute it and/or modify
7 - it under the terms of the GNU General Public License version 3 as 7 - it under the terms of the GNU General Public License version 3 as
8 - published by the Free Software Foundation. 8 - published by the Free Software Foundation.
9 - 9 -
10 - Adblock Plus is distributed in the hope that it will be useful, 10 - Adblock Plus is distributed in the hope that it will be useful,
11 - but WITHOUT ANY WARRANTY; without even the implied warranty of 11 - but WITHOUT ANY WARRANTY; without even the implied warranty of
12 - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 12 - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 - GNU General Public License for more details. 13 - GNU General Public License for more details.
14 - 14 -
15 - You should have received a copy of the GNU General Public License 15 - You should have received a copy of the GNU General Public License
16 - along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. 16 - along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
17 --> 17 -->
18 18
19 <html> 19 <html>
20 <head> 20 <head>
21 <title class="i18n_firstRun_title"></title> 21 <title class="i18n_firstRun_title"></title>
22 <meta charset="utf-8"> 22 <meta charset="utf-8">
23
24 <!-- {% if type == 'safari' %} -->
Felix Dahlke 2013/10/15 10:31:23 Why are the Jinja tags in HTML comments? I presume
Sebastian Noack 2013/10/18 14:40:41 See the discussion in the other review. http://co
25 <!-- {{ '<link type="text/css" href="skin/firstRun.css" rel="stylesheet"/>' }} -->
26 <!-- {% else %} -->
Felix Dahlke 2013/10/15 10:31:23 How about something shorter like this instead? <l
Sebastian Noack 2013/10/18 14:40:41 That wouldn't work, because for compatibility jinj
23 <link type="text/css" href="/skin/firstRun.css" rel="stylesheet"/> 27 <link type="text/css" href="/skin/firstRun.css" rel="stylesheet"/>
28 <!-- {% endif %} -->
29
30 <!-- {% if type in ('chrome', 'opera', 'safari') %} -->
31 <!-- {{ '<script type="text/javascript" src="ext.js"></script>' }} -->
32 <!-- {% endif %} -->
24 <script type="text/javascript" src="utils.js"></script> 33 <script type="text/javascript" src="utils.js"></script>
25 <script type="text/javascript" src="i18n.js"></script> 34 <script type="text/javascript" src="i18n.js"></script>
26 <script type="text/javascript" src="firstRun.js"></script> 35 <script type="text/javascript" src="firstRun.js"></script>
27 </head> 36 </head>
28 <body> 37 <body>
29 <div id="shade" class="arrow"></div> 38 <div id="shade" class="arrow"></div>
30 <div id="content"> 39 <div id="content">
31 <header> 40 <header>
32 <h1 id="title-main" class="i18n_firstRun_title"></h1> 41 <h1 id="title-main" class="i18n_firstRun_title"></h1>
33 </header> 42 </header>
34 43
44 <p id="legacySafariWarning" class="i18n_firstRun_legacySafariWarning" hidd en="true"></p>
35 <p id="dataCorruptionWarning" class="i18n_firstRun_dataCorruptionWarning" hidden="true"></p> 45 <p id="dataCorruptionWarning" class="i18n_firstRun_dataCorruptionWarning" hidden="true"></p>
36
37 <p id="acceptableAdsExplanation" class="i18n_firstRun_acceptableAdsExplana tion"></p> 46 <p id="acceptableAdsExplanation" class="i18n_firstRun_acceptableAdsExplana tion"></p>
38 47
39 <h2 id="features-title" class="i18n_firstRun_features"></h2> 48 <h2 id="features-title" class="i18n_firstRun_features"></h2>
40 49
41 <ul id="features"> 50 <ul id="features">
42 <li id="feature-malware" class="feature"> 51 <li id="feature-malware" class="feature">
43 <div class="feature-image"></div> 52 <div class="feature-image"></div>
44 <div class="feature-description"> 53 <div class="feature-description">
45 <p class="feature-text"> 54 <p class="feature-text">
46 <span class="i18n_firstRun_feature_malware feature-title"></span> <span class="i18n_firstRun_feature_malware_description"></span> 55 <span class="i18n_firstRun_feature_malware feature-title"></span> <span class="i18n_firstRun_feature_malware_description"></span>
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 </li> 103 </li>
95 </ul> 104 </ul>
96 </footer> 105 </footer>
97 </div> 106 </div>
98 107
99 <div id="glass-pane"> 108 <div id="glass-pane">
100 <iframe id="share-popup" scrolling="no"></iframe> 109 <iframe id="share-popup" scrolling="no"></iframe>
101 </div> 110 </div>
102 </body> 111 </body>
103 </html> 112 </html>
OLDNEW
« no previous file with comments | « no previous file | chrome/content/ui/firstRun.js » ('j') | chrome/content/ui/firstRun.js » ('J')

Powered by Google App Engine
This is Rietveld