From b0012d2aeb6a5ef8172223c58ceb8fa5c78c5138 Mon Sep 17 00:00:00 2001 From: jrobson Date: Sun, 11 May 2014 16:17:05 +0200 Subject: [PATCH] Simplify thumbnail styling to have consistency with mobile implementation Remove undocumented !important rule - !important is bad. Bug: 65160 Change-Id: I21d6856d51478395cbe59e86e2f3aec495a74a38 --- skins/common/commonContent.css | 2 +- skins/vector/components/thumbnails.less | 32 +++++++++++++++++++++++++ skins/vector/screen.less | 1 + 3 files changed, 34 insertions(+), 1 deletion(-) create mode 100644 skins/vector/components/thumbnails.less diff --git a/skins/common/commonContent.css b/skins/common/commonContent.css index 4157b55b8c..6bf01ae21c 100644 --- a/skins/common/commonContent.css +++ b/skins/common/commonContent.css @@ -143,7 +143,7 @@ html .thumbimage { html .thumbcaption { border: none; line-height: 1.4em; - padding: 3px !important; + padding: 3px; font-size: 94%; } diff --git a/skins/vector/components/thumbnails.less b/skins/vector/components/thumbnails.less new file mode 100644 index 0000000000..afb5f2a929 --- /dev/null +++ b/skins/vector/components/thumbnails.less @@ -0,0 +1,32 @@ +// FIXME: Due to commonContent.css we have to use html tag to make sure these are more specific +html .thumb { + &.tright { + padding: 1px 0 0 15px; + } + + &.tleft { + padding: 1px 15px 0 0; + } + + // these resets should be unnecessary + div.thumbinner { + border: none; + background: none; + font-size: 100%; + } + + .thumbcaption { + line-height: @content-line-height; + font-size: .85em; + padding: 3px 0 0 0; + color: #333; + } + + .magnify { + display: none; + } + + .thumbimage { + border: none; + } +} diff --git a/skins/vector/screen.less b/skins/vector/screen.less index f7b374f5be..3e824f33b8 100644 --- a/skins/vector/screen.less +++ b/skins/vector/screen.less @@ -8,3 +8,4 @@ @import "components/footer.less"; @import 'components/notifications.less'; @import "components/externalLinks.less"; +@import "components/thumbnails.less"; -- 2.20.1