r112513 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r112512‎ | r112513 | r112514 >
Date:20:39, 27 February 2012
Author:hashar
Status:ok
Tags:
Comment:
MFT to 1.19wmf1 r112510 r112511 r112512

Special:CodeReview:

* (bug 27375) Whitespace changes should be identifiable in CR diffs
* Remove text-decoration on td.ins
Modified paths:
  • /branches/wmf/1.19wmf1/extensions/CodeReview/modules/ext.codereview.styles.css (modified) (history)

Diff [purge]

Index: branches/wmf/1.19wmf1/extensions/CodeReview/modules/ext.codereview.styles.css
@@ -11,16 +11,19 @@
1212 .TablePager_col_comments {
1313 text-align: center;
1414 }
 15+
1516 .TablePager_col_cr_message {
1617 max-width: 20em;
1718 overflow: hidden;
1819 }
 20+
1921 /* @noflip */
2022 .sitedir-ltr .TablePager_col_cr_message,
2123 .sitedir-ltr .TablePager_col_cc_text,
2224 .sitedir-ltr .mw-codereview-message {
2325 direction: ltr;
2426 }
 27+
2528 /* @noflip */
2629 .sitedir-rtl .TablePager_col_cr_message,
2730 .sitedir-rtl .TablePager_col_cc_text,
@@ -72,34 +75,41 @@
7376 .mw-codereview-status-fixme td {
7477 background: #ff9999 !important;
7578 }
 79+
7680 .mw-codereview-status-fixme:hover,
7781 .mw-codereview-status-fixme:hover td {
7882 background: #df0000 !important;
7983 color: white;
8084 }
 85+
8186 .mw-codereview-status-fixme:hover a,
8287 .mw-codereview-status-fixme:hover td a {
8388 color: #ff0 !important;
8489 }
 90+
8591 .mw-codereview-status-resolved,
8692 .mw-codereview-status-resolved td {
8793 background: #c0ffc0 !important;
8894 }
 95+
8996 .mw-codereview-status-resolved:hover,
9097 .mw-codereview-status-resolved:hover td {
9198 background: #a0dfa0 !important;
9299 }
 100+
93101 .mw-codereview-status-reverted,
94102 .mw-codereview-status-reverted td {
95103 background: #ddd !important;
96104 color: #666 !important;
97105 text-decoration: line-through !important;
98106 }
 107+
99108 .mw-codereview-status-reverted:hover,
100109 .mw-codereview-status-reverted:hover td {
101110 background: #aaa !important;
102111 text-decoration: line-through !important;
103112 }
 113+
104114 .mw-codereview-status-deferred,
105115 .mw-codereview-status-deferred td {
106116 color: #666;
@@ -126,6 +136,7 @@
127137 .mw-codereview-success {
128138 color: #1a2;
129139 }
 140+
130141 .mw-codereview-fail {
131142 color: #d21;
132143 }
@@ -148,6 +159,7 @@
149160 background-color: #EEE;
150161 color: black;
151162 }
 163+
152164 .mw-codereview-diff tr.patchedfile td {
153165 padding: 1em;
154166 border: 1px solid #CCC;
@@ -169,14 +181,25 @@
170182 -o-user-select: none;
171183 user-select: none;
172184 }
 185+
173186 .mw-codereview-diff td.ins {
174 - text-decoration: none;
175187 color: green;
176188 }
 189+
177190 .mw-codereview-diff td.del {
178 - text-decoration: none;
179191 color: red;
180192 }
 193+
 194+.mw-codereview-diff td.ins ins {
 195+ background: #EDFFED;
 196+ text-decoration: none;
 197+}
 198+
 199+.mw-codereview-diff td.del del {
 200+ background: #FFEDED;
 201+ text-decoration: none;
 202+}
 203+
181204 .mw-codereview-diff td.chunkdelimiter {
182205 background-color: #EDEDFF;
183206 color: black;
@@ -193,11 +216,16 @@
194217 width: 100%;
195218 }
196219
197 -.mw-wordcloud-size-1, .mw-wordcloud-size-2,
198 -.mw-wordcloud-size-3, .mw-wordcloud-size-4,
199 -.mw-wordcloud-size-5, .mw-wordcloud-size-6,
200 -.mw-wordcloud-size-7, .mw-wordcloud-size-8,
201 -.mw-wordcloud-size-9, .mw-wordcloud-size-0 {
 220+.mw-wordcloud-size-1,
 221+.mw-wordcloud-size-2,
 222+.mw-wordcloud-size-3,
 223+.mw-wordcloud-size-4,
 224+.mw-wordcloud-size-5,
 225+.mw-wordcloud-size-6,
 226+.mw-wordcloud-size-7,
 227+.mw-wordcloud-size-8,
 228+.mw-wordcloud-size-9,
 229+.mw-wordcloud-size-0 {
202230 padding: 4px;
203231 letter-spacing: 0.1em;
204232 }
@@ -206,38 +234,47 @@
207235 color: #222;
208236 font-size: 240%;
209237 }
 238+
210239 a.mw-wordcloud-size-2 {
211240 color: #333;
212241 font-size:220%;
213242 }
 243+
214244 a.mw-wordcloud-size-3 {
215245 color: #444;
216246 font-size: 200%;
217247 }
 248+
218249 a.mw-wordcloud-size-4 {
219250 color: #555;
220251 font-size: 180%;
221252 }
 253+
222254 a.mw-wordcloud-size-5 {
223255 color: #666;
224256 font-size: 160%;
225257 }
 258+
226259 a.mw-wordcloud-size-6 {
227260 color: #777;
228261 font-size: 140%;
229262 }
 263+
230264 a.mw-wordcloud-size-7 {
231265 color: #888;
232266 font-size: 120%;
233267 }
 268+
234269 a.mw-wordcloud-size-8 {
235270 color: #999;
236271 font-size: 100%;
237272 }
 273+
238274 a.mw-wordcloud-size-9 {
239275 color: #aaa;
240276 font-size: 90%;
241277 }
 278+
242279 a.mw-wordcloud-size-10,
243280 a.mw-wordcloud-size-0 {
244281 color: #bbb;

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r82064* (bug 27375) Do the diff colouring in CRreedy18:28, 13 February 2011
r84818Revert r82064, fixes bug 27416, reopens bug 27375reedy19:34, 26 March 2011
r112510[ext.codereview.styles] Whitespacekrinkle20:26, 27 February 2012
r112511[ext.codereview.styles] Make whitespace changes identifiable in CR diffs...krinkle20:30, 27 February 2012
r112512[ext.codereview.styles] follow-up r112511: spaces to tabskrinkle20:33, 27 February 2012

Status & tagging log