r87012 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r87011‎ | r87012 | r87013 >
Date:14:37, 27 April 2011
Author:catrope
Status:ok
Tags:
Comment:
CodeReview: (bug 28703) Don't load styles with JS, because they're applied to HTML generated server-side
Modified paths:
  • /trunk/extensions/CodeReview/CodeReview.php (modified) (history)
  • /trunk/extensions/CodeReview/modules/ext.codereview.css (deleted) (history)
  • /trunk/extensions/CodeReview/modules/ext.codereview.styles.css (added) (history)
  • /trunk/extensions/CodeReview/ui/SpecialCode.php (modified) (history)

Diff [purge]

Index: trunk/extensions/CodeReview/CodeReview.php
——@@ -143,11 +143,14 @@
144144
145145 // Styles and any code common to all Special:Code subviews:
146146 $wgResourceModules['ext.codereview'] = array(
147 - 'styles' => 'ext.codereview.css',
148147 'scripts' => 'ext.codereview.js',
149148 'dependencies' => 'jquery.suggestions',
150149 ) + $commonModuleInfo;
151150
 151+$wgResourceModules['ext.codereview.styles'] = array(
 152+ 'styles' => 'ext.codereview.styles.css',
 153+) + $commonModuleInfo;
 154+
152155 // On-demand diff loader for CodeRevisionView:
153156 $wgResourceModules['ext.codereview.loaddiff'] = array(
154157 'scripts' => 'ext.codereview.loaddiff.js'
Index: trunk/extensions/CodeReview/modules/ext.codereview.css
——@@ -1,193 +0,0 @@
2 -table.mw-codereview-meta td {
3 - vertical-align: top;
4 -}
5 -
6 -.mw-codereview-paths {
7 - max-height:600px;
8 - overflow:auto;
9 -}
10 -
11 -.TablePager_col_selectforchange,
12 -.TablePager_col_comments {
13 - text-align: center;
14 -}
15 -.TablePager_col_cr_message {
16 - max-width: 20em;
17 - overflow: hidden;
18 -}
19 -
20 -.TablePager_col_cr_timestamp {
21 - white-space: nowrap;
22 -}
23 -
24 -.mw-codereview-comment {
25 - border: solid 1px #aaaab3;
26 - padding: 0;
27 - background: #eee;
28 - margin-bottom: 8px;
29 -}
30 -
31 -.mw-codereview-imgdiff {
32 - border: solid 1px #aaaab3;
33 - padding: 1em;
34 - background: #eee;
35 -}
36 -
37 -.mw-codereview-comment-meta {
38 - padding: 4px 8px;
39 -}
40 -
41 -.mw-codereview-comment-text {
42 - background: white;
43 - padding: 16px;
44 -}
45 -
46 -.mw-codereview-comment .mw-codereview-post-comment {
47 - padding: 16px 16px 16px 48px;
48 -}
49 -
50 -.mw-codereview-status-new,
51 -.mw-codereview-status-new td {
52 - background: #ffffc0 !important;
53 -}
54 -
55 -.mw-codereview-status-new:hover,
56 -.mw-codereview-status-new:hover td {
57 - background: #dfdfa0 !important;
58 -}
59 -
60 -.mw-codereview-status-fixme,
61 -.mw-codereview-status-fixme td {
62 - background: #ff9999 !important;
63 -}
64 -.mw-codereview-status-fixme:hover,
65 -.mw-codereview-status-fixme:hover td {
66 - background: #df0000 !important;
67 - color: white;
68 -}
69 -.mw-codereview-status-fixme:hover a,
70 -.mw-codereview-status-fixme:hover td a {
71 - color: #ff0 !important;
72 -}
73 -
74 -.mw-codereview-status-resolved,
75 -.mw-codereview-status-resolved td {
76 - background: #c0ffc0 !important;
77 -}
78 -.mw-codereview-status-resolved:hover,
79 -.mw-codereview-status-resolved:hover td {
80 - background: #a0dfa0 !important;
81 -}
82 -
83 -.mw-codereview-status-reverted,
84 -.mw-codereview-status-reverted td {
85 - background: #ddd !important;
86 - color: #666 !important;
87 - text-decoration: line-through !important;
88 -}
89 -.mw-codereview-status-reverted:hover,
90 -.mw-codereview-status-reverted:hover td {
91 - background: #aaa !important;
92 - text-decoration: line-through !important;
93 -}
94 -.mw-codereview-status-deferred,
95 -.mw-codereview-status-deferred td {
96 - color: #666;
97 -}
98 -
99 -.mw-codereview-status-old,
100 -.mw-codereview-status-old td {
101 - color: #666;
102 -}
103 -
104 -.mw-codereview-associateform {
105 - float: right;
106 -}
107 -
108 -.mw-codereview-signoffchecks {
109 - float: right;
110 -}
111 -
112 -.mw-codereview-struck td {
113 - text-decoration: line-through;
114 - background: #aaa;
115 -}
116 -
117 -.mw-codereview-success {
118 - color: #1a2;
119 -}
120 -.mw-codereview-fail {
121 - color: #d21;
122 -}
123 -
124 -/* Diffs */
125 -.mw-codereview-diff ins {
126 - text-decoration: none;
127 - color: green;
128 -}
129 -
130 -.mw-codereview-diff del {
131 - text-decoration: none;
132 - color: red;
133 -}
134 -
135 -.mw-codereview-diff .meta {
136 - color: #008b8b;
137 -}
138 -
139 -/* WordCloud for tag list view */
140 -
141 -div.mw-wordcloud {
142 - width: 100%;
143 -}
144 -
145 -.mw-wordcloud-size-1, .mw-wordcloud-size-2,
146 -.mw-wordcloud-size-3, .mw-wordcloud-size-4,
147 -.mw-wordcloud-size-5, .mw-wordcloud-size-6,
148 -.mw-wordcloud-size-7, .mw-wordcloud-size-8,
149 -.mw-wordcloud-size-9, .mw-wordcloud-size-0 {
150 - padding: 4px;
151 - letter-spacing: 0.1em;
152 -}
153 -
154 -a.mw-wordcloud-size-1 {
155 - color: #222;
156 - font-size: 240%;
157 -}
158 -a.mw-wordcloud-size-2 {
159 - color: #333;
160 - font-size:220%;
161 -}
162 -a.mw-wordcloud-size-3 {
163 - color: #444;
164 - font-size: 200%;
165 -}
166 -a.mw-wordcloud-size-4 {
167 - color: #555;
168 - font-size: 180%;
169 -}
170 -a.mw-wordcloud-size-5 {
171 - color: #666;
172 - font-size: 160%;
173 -}
174 -a.mw-wordcloud-size-6 {
175 - color: #777;
176 - font-size: 140%;
177 -}
178 -a.mw-wordcloud-size-7 {
179 - color: #888;
180 - font-size: 120%;
181 -}
182 -a.mw-wordcloud-size-8 {
183 - color: #999;
184 - font-size: 100%;
185 -}
186 -a.mw-wordcloud-size-9 {
187 - color: #aaa;
188 - font-size: 90%;
189 -}
190 -a.mw-wordcloud-size-10,
191 -a.mw-wordcloud-size-0 {
192 - color: #bbb;
193 - font-size: 80%;
194 -}
Index: trunk/extensions/CodeReview/modules/ext.codereview.styles.css
——@@ -0,0 +1,193 @@
 2+table.mw-codereview-meta td {
 3+ vertical-align: top;
 4+}
 5+
 6+.mw-codereview-paths {
 7+ max-height:600px;
 8+ overflow:auto;
 9+}
 10+
 11+.TablePager_col_selectforchange,
 12+.TablePager_col_comments {
 13+ text-align: center;
 14+}
 15+.TablePager_col_cr_message {
 16+ max-width: 20em;
 17+ overflow: hidden;
 18+}
 19+
 20+.TablePager_col_cr_timestamp {
 21+ white-space: nowrap;
 22+}
 23+
 24+.mw-codereview-comment {
 25+ border: solid 1px #aaaab3;
 26+ padding: 0;
 27+ background: #eee;
 28+ margin-bottom: 8px;
 29+}
 30+
 31+.mw-codereview-imgdiff {
 32+ border: solid 1px #aaaab3;
 33+ padding: 1em;
 34+ background: #eee;
 35+}
 36+
 37+.mw-codereview-comment-meta {
 38+ padding: 4px 8px;
 39+}
 40+
 41+.mw-codereview-comment-text {
 42+ background: white;
 43+ padding: 16px;
 44+}
 45+
 46+.mw-codereview-comment .mw-codereview-post-comment {
 47+ padding: 16px 16px 16px 48px;
 48+}
 49+
 50+.mw-codereview-status-new,
 51+.mw-codereview-status-new td {
 52+ background: #ffffc0 !important;
 53+}
 54+
 55+.mw-codereview-status-new:hover,
 56+.mw-codereview-status-new:hover td {
 57+ background: #dfdfa0 !important;
 58+}
 59+
 60+.mw-codereview-status-fixme,
 61+.mw-codereview-status-fixme td {
 62+ background: #ff9999 !important;
 63+}
 64+.mw-codereview-status-fixme:hover,
 65+.mw-codereview-status-fixme:hover td {
 66+ background: #df0000 !important;
 67+ color: white;
 68+}
 69+.mw-codereview-status-fixme:hover a,
 70+.mw-codereview-status-fixme:hover td a {
 71+ color: #ff0 !important;
 72+}
 73+
 74+.mw-codereview-status-resolved,
 75+.mw-codereview-status-resolved td {
 76+ background: #c0ffc0 !important;
 77+}
 78+.mw-codereview-status-resolved:hover,
 79+.mw-codereview-status-resolved:hover td {
 80+ background: #a0dfa0 !important;
 81+}
 82+
 83+.mw-codereview-status-reverted,
 84+.mw-codereview-status-reverted td {
 85+ background: #ddd !important;
 86+ color: #666 !important;
 87+ text-decoration: line-through !important;
 88+}
 89+.mw-codereview-status-reverted:hover,
 90+.mw-codereview-status-reverted:hover td {
 91+ background: #aaa !important;
 92+ text-decoration: line-through !important;
 93+}
 94+.mw-codereview-status-deferred,
 95+.mw-codereview-status-deferred td {
 96+ color: #666;
 97+}
 98+
 99+.mw-codereview-status-old,
 100+.mw-codereview-status-old td {
 101+ color: #666;
 102+}
 103+
 104+.mw-codereview-associateform {
 105+ float: right;
 106+}
 107+
 108+.mw-codereview-signoffchecks {
 109+ float: right;
 110+}
 111+
 112+.mw-codereview-struck td {
 113+ text-decoration: line-through;
 114+ background: #aaa;
 115+}
 116+
 117+.mw-codereview-success {
 118+ color: #1a2;
 119+}
 120+.mw-codereview-fail {
 121+ color: #d21;
 122+}
 123+
 124+/* Diffs */
 125+.mw-codereview-diff ins {
 126+ text-decoration: none;
 127+ color: green;
 128+}
 129+
 130+.mw-codereview-diff del {
 131+ text-decoration: none;
 132+ color: red;
 133+}
 134+
 135+.mw-codereview-diff .meta {
 136+ color: #008b8b;
 137+}
 138+
 139+/* WordCloud for tag list view */
 140+
 141+div.mw-wordcloud {
 142+ width: 100%;
 143+}
 144+
 145+.mw-wordcloud-size-1, .mw-wordcloud-size-2,
 146+.mw-wordcloud-size-3, .mw-wordcloud-size-4,
 147+.mw-wordcloud-size-5, .mw-wordcloud-size-6,
 148+.mw-wordcloud-size-7, .mw-wordcloud-size-8,
 149+.mw-wordcloud-size-9, .mw-wordcloud-size-0 {
 150+ padding: 4px;
 151+ letter-spacing: 0.1em;
 152+}
 153+
 154+a.mw-wordcloud-size-1 {
 155+ color: #222;
 156+ font-size: 240%;
 157+}
 158+a.mw-wordcloud-size-2 {
 159+ color: #333;
 160+ font-size:220%;
 161+}
 162+a.mw-wordcloud-size-3 {
 163+ color: #444;
 164+ font-size: 200%;
 165+}
 166+a.mw-wordcloud-size-4 {
 167+ color: #555;
 168+ font-size: 180%;
 169+}
 170+a.mw-wordcloud-size-5 {
 171+ color: #666;
 172+ font-size: 160%;
 173+}
 174+a.mw-wordcloud-size-6 {
 175+ color: #777;
 176+ font-size: 140%;
 177+}
 178+a.mw-wordcloud-size-7 {
 179+ color: #888;
 180+ font-size: 120%;
 181+}
 182+a.mw-wordcloud-size-8 {
 183+ color: #999;
 184+ font-size: 100%;
 185+}
 186+a.mw-wordcloud-size-9 {
 187+ color: #aaa;
 188+ font-size: 90%;
 189+}
 190+a.mw-wordcloud-size-10,
 191+a.mw-wordcloud-size-0 {
 192+ color: #bbb;
 193+ font-size: 80%;
 194+}
Property changes on: trunk/extensions/CodeReview/modules/ext.codereview.styles.css
___________________________________________________________________
Added: svn:eol-style
1195 + native
Index: trunk/extensions/CodeReview/ui/SpecialCode.php
——@@ -20,6 +20,7 @@
2121 // Base styles used for all code review UI actions.
2222 $wgOut->addModules( 'ext.codereview' );
2323 $wgOut->addModules( 'ext.codereview.tooltips' );
 24+ $wgOut->addModuleStyles( 'ext.codereview.styles' );
2425
2526 $view = $this->getViewFrom( $subpage );
2627 if( $view ) {

Follow-up revisions

RevisionCommit summaryAuthorDate
r87090MFT a few minor CR fixes (r86477, r86478, r86479, r87012)demon18:11, 28 April 2011

Status & tagging log