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

Side by Side Diff: pages/subscriptions.tmpl

Issue 29337807: Issue 3097 - Refactored subscriptions list table on adblockplus.org/en/subscriptions (Closed)
Patch Set: Changed table into unordered list, refactored list markup, refactored main css and templates to support multiple footer elements Created Feb. 27, 2016, 4:16 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
OLDNEW
1 title=Known Adblock Plus subscriptions 1 title=Known Adblock Plus subscriptions
2 {# 2 {#
3 # This file is part of the Adblock Plus website, 3 # This file is part of the Adblock Plus website,
4 # Copyright (C) 2006-2016 Eyeo GmbH 4 # Copyright (C) 2006-2016 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 19
20 <style type="text/css"> 20 <style type="text/css">
21 .subscriptions 21 .subscriptions-list,
22 .supplemented-subscriptions-list
22 { 23 {
23 border-collapse: collapse; 24 margin: 0;
Thomas Greiner 2016/07/15 16:07:54 Coding style: "CSS number values should specify un
juliandoucette 2016/07/19 22:29:17 Done.
24 width: 100%; 25 list-style: none;
25 } 26 }
26 27
27 .subscriptions td 28 .subscriptions-list
28 { 29 {
30 padding: 0;
31 }
32
33 .supplemented-subscriptions-list
34 {
35 padding-left: 30px;
Thomas Greiner 2016/07/15 16:07:54 Please avoid using "border/margin/padding-bottom/l
juliandoucette 2016/07/19 22:29:17 I hadn't considered rtl when I created this patch.
36 }
37
38 .subscription
39 {
40 position: relative;
41 overflow: auto;
42 /* Using outline and margin instead of border to prevent overlapping
43 and missing connections in hirarchy */
Thomas Greiner 2016/07/15 16:07:54 Typo: Replace "hirarchy" with "hierarchy".
juliandoucette 2016/07/19 22:29:17 Thanks! :)
44 outline: 1px dashed #000;
45 margin-left: -1px;
46 margin-top: 1px;
47 }
48
49 .subscription h1
50 {
51 margin: 0;
52 font-size: 16px;
53 }
54
55 .subscription p
56 {
57 margin: 0;
58 }
59
60 .subscription-summary,
61 .subscription-authors,
62 .subscription-links
63 {
64 padding: 5px;
65 }
66
67 .subscription-content,
68 .subscription-details
69 {
70 float: left;
29 width: 50%; 71 width: 50%;
30 } 72 }
31 73
32 .subscriptions td, .subscriptions th 74
75 .subscription-authors,
76 .subscription-links
33 { 77 {
34 vertical-align: top; 78 border-left: 1px dashed #000;
35 border: 1px dashed black;
36 padding: 5px;
37 } 79 }
38 80
39 .subscriptions .dummy 81 .subscription-links
40 { 82 {
41 border-style: none; 83 border-top: 1px dashed #000;
42 width: 20px;
43 min-width: 20px;
44 } 84 }
45 85
46 .subscriptions .deprecated 86 .subscriptions .deprecated
47 { 87 {
48 opacity: 0.5; 88 opacity: 0.5;
49 } 89 }
50 </style> 90 </style>
51 91
52 <p>{{"This is a list of the actively maintained filter subscriptions we're aware of. You can subscribe to any of them by simply clicking its \"Subscribe\" link and confirming your decision. The filters will be added to Adblock Plus and upda ted automatically. Note that <a href=\"getting_started#subscription\">you should n't add too many filter subscriptions</a>."|translate("s1")}}</p> 92 <p>{{"This is a list of the actively maintained filter subscriptions we're aware of. You can subscribe to any of them by simply clicking its \"Subscribe\" link and confirming your decision. The filters will be added to Adblock Plus and upda ted automatically. Note that <a href=\"getting_started#subscription\">you should n't add too many filter subscriptions</a>."|translate("s1")}}</p>
53 93
54 94
55 {% from "includes/subscriptionList" import display_subscriptions %} 95 {% from "includes/subscriptionList" import display_subscriptions %}
56 96
57 {{ display_subscriptions(1|get_subscriptions) }} 97 {{ display_subscriptions(1|get_subscriptions) }}
58 98
59 99
60 <p>{{"Can't see your filter list here? <a href=\"mailto:subscriptionlist@adblock plus.org\">Send</a> the information you would like to be displayed here. Please note that in order to be included in this list your subscription must contain a comment at the beginning describing the origin of the filter list and the prefer red way of contacting you if problems with your list occur. You should also be c ertain that when these issues arise you will be able to fix them in a reasonable amount of time."|translate("s2")}}</p> 100 <p>{{"Can't see your filter list here? <a href=\"mailto:subscriptionlist@adblock plus.org\">Send</a> the information you would like to be displayed here. Please note that in order to be included in this list your subscription must contain a comment at the beginning describing the origin of the filter list and the prefer red way of contacting you if problems with your list occur. You should also be c ertain that when these issues arise you will be able to fix them in a reasonable amount of time."|translate("s2")}}</p>
61 101
62 <p> 102 <p>
63 {{"Please note that all of these filter lists are being maintained by third part ies. Eyeo, the company behind Adblock Plus, has no control over them.\nWe are on ly listing them to help you find lists that could be useful to you.\nIn no case shall Eyeo be held liable towards you or any other third party for any loss or d amage (including but not limited to loss of chances and of business) deriving di rectly or indirectly from your use of these lists."|translate("s3")}} 103 {{"Please note that all of these filter lists are being maintained by third part ies. Eyeo, the company behind Adblock Plus, has no control over them.\nWe are on ly listing them to help you find lists that could be useful to you.\nIn no case shall Eyeo be held liable towards you or any other third party for any loss or d amage (including but not limited to loss of chances and of business) deriving di rectly or indirectly from your use of these lists."|translate("s3")}}
64 </p> 104 </p>
OLDNEW

Powered by Google App Engine
This is Rietveld