r99758 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r99757‎ | r99758 | r99759 >
Date:08:00, 14 October 2011
Author:reedy
Status:ok (Comments)
Tags:
Comment:
Bug 31050 - Cleanup duplicate CSS for mobile devices

More gone!
Modified paths:
  • /trunk/extensions/MobileFrontend/stylesheets/android.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/blackberry.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/common.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/default.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/firefox.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/iphone.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/iphone2.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/kindle.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/nokia.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/operamini.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/palm_pre.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/psp.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/simple.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/webkit.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/webkit_old.css (modified) (history)
  • /trunk/extensions/MobileFrontend/stylesheets/wii.css (modified) (history)

Diff [purge]

Index: trunk/extensions/MobileFrontend/stylesheets/wii.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,11 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
7454 table.gallery .gallerytext {
7555 margin-top: -12px;
7656 text-align: center;
Index: trunk/extensions/MobileFrontend/stylesheets/common.css
@@ -93,3 +93,29 @@
9494 font-size: 0.9em;
9595 width: 100% !important;
9696 }
 97+
 98+table.gallery .gallerytext {
 99+ margin-top: -12px;
 100+ text-align: center;
 101+}
 102+
 103+table.navbox th.navbox-title {
 104+ background: #ccccff;
 105+ padding: 5px;
 106+}
 107+
 108+table.navbox div[style*="padding"] {
 109+ padding-left: 0 !important;
 110+ padding-right: 0 !important;
 111+}
 112+
 113+table.gallery .gallerybox[style] {
 114+ width: auto !important;
 115+ margin-bottom: -16px;
 116+}
 117+
 118+table td, table th {
 119+ border: 1px solid #cccccc;
 120+ padding: 3px;
 121+}
 122+
Index: trunk/extensions/MobileFrontend/stylesheets/simple.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/palm_pre.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/nokia.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/firefox.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/psp.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/iphone2.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/kindle.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/android.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/blackberry.css
@@ -2,11 +2,6 @@
33 test: 1;
44 }
55
6 -table td, table th {
7 - border: 1px solid #cccccc;
8 - padding: 3px;
9 -}
10 -
116 table table {
127 border: 0;
138 }
@@ -19,11 +14,6 @@
2015 background: #f9f9f9;
2116 }
2217
23 -table.navbox th.navbox-title {
24 - background: #ccccff;
25 - padding: 5px;
26 -}
27 -
2818 table.navbox td.navbox-abovebelow {
2919 padding: 3px;
3020 }
@@ -32,11 +22,6 @@
3323 padding: 3px;
3424 }
3525
36 -table.navbox div[style*="padding"] {
37 - padding-left: 0 !important;
38 - padding-right: 0 !important;
39 -}
40 -
4126 table.navbox div[style*="padding"] a {
4227 white-space: pre-wrap;
4328 }
@@ -69,16 +54,6 @@
7055 border: 1px solid #cccccc;
7156 }
7257
73 -table.gallery .gallerybox[style] {
74 - width: auto !important;
75 - margin-bottom: -16px;
76 -}
77 -
78 -table.gallery .gallerytext {
79 - margin-top: -12px;
80 - text-align: center;
81 -}
82 -
8358 table.geography[style] th {
8459 text-align: left;
8560 }
Index: trunk/extensions/MobileFrontend/stylesheets/iphone.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/operamini.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/webkit_old.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/default.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }
Index: trunk/extensions/MobileFrontend/stylesheets/webkit.css
@@ -1,8 +1,3 @@
2 -table td, table th {
3 - border: 1px solid #cccccc;
4 - padding: 3px;
5 -}
6 -
72 table table {
83 border: 0;
94 }
@@ -15,11 +10,6 @@
1611 background: #f9f9f9;
1712 }
1813
19 -table.navbox th.navbox-title {
20 - background: #ccccff;
21 - padding: 5px;
22 -}
23 -
2414 table.navbox td.navbox-abovebelow {
2515 padding: 3px;
2616 }
@@ -28,11 +18,6 @@
2919 padding: 3px;
3020 }
3121
32 -table.navbox div[style*="padding"] {
33 - padding-left: 0 !important;
34 - padding-right: 0 !important;
35 -}
36 -
3722 table.navbox div[style*="padding"] a {
3823 white-space: pre-wrap;
3924 }
@@ -65,16 +50,6 @@
6651 border: 1px solid #cccccc;
6752 }
6853
69 -table.gallery .gallerybox[style] {
70 - width: auto !important;
71 - margin-bottom: -16px;
72 -}
73 -
74 -table.gallery .gallerytext {
75 - margin-top: -12px;
76 - text-align: center;
77 -}
78 -
7954 table.geography[style] th {
8055 text-align: left;
8156 }

Follow-up revisions

RevisionCommit summaryAuthorDate
r99826mft r99758preilly23:06, 14 October 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r99656Add common.css...reedy23:27, 12 October 2011
r99657Part of Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy23:32, 12 October 2011
r99658Part of Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy23:36, 12 October 2011
r99659Part of Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy23:43, 12 October 2011
r99660Part of Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy23:50, 12 October 2011
r99661Part of Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy23:59, 12 October 2011
r99719Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy19:18, 13 October 2011
r99721Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy19:33, 13 October 2011
r99723Bug 31050 - Cleanup duplicate CSS for mobile devices...reedy19:51, 13 October 2011

Comments

#Comment by G.Hagedorn (talk | contribs)   08:10, 16 October 2011

tag for 1.18?

Status & tagging log