Arlolra has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/366898 )
Change subject: Display border-bottom when figcaption isn't present
......................................................................
Display border-bottom when figcaption isn't present
* Parsoid doesn't emit a figcaption when no caption is present, so
having the border-bottom styled on it meant there are cases where it
goes missing.
Change-Id: I2ff8e41d3cb973f56f6571d84fb662aed23b1f10
---
M resources/src/mediawiki.skinning/content.parsoid.less
1 file changed, 19 insertions(+), 36 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/98/366898/1
diff --git a/resources/src/mediawiki.skinning/content.parsoid.less
b/resources/src/mediawiki.skinning/content.parsoid.less
index 7a7df7a..cf32055 100644
--- a/resources/src/mediawiki.skinning/content.parsoid.less
+++ b/resources/src/mediawiki.skinning/content.parsoid.less
@@ -86,27 +86,9 @@
float: none;
}
+ /* Hide the caption for frameless and plain floated images */
> figcaption {
- display: table-caption;
- caption-side: bottom;
- /* In mw-core the font-size is duplicated, 94% in thumbiner
- * and again 94% in thumbcaption. 88.4% for font size of the
- * caption results in the same behavior. */
- font-size: 88.4%;
- line-height: 1.4em;
- text-align: left;
-
- border: 1px solid #c8ccd1;
- border-top: 0;
-
- /* taken from .thumbcaption, plus .thumbinner */
- padding: 0 6px 6px 6px;
- background-color: #f8f9fa;
-
- table {
- /* reset caption side for tables inside figcaptions */
- caption-side: top;
- }
+ display: none;
}
}
@@ -117,12 +99,12 @@
figure[typeof~='mw:Video/Frame'],
figure[typeof~='mw:Audio/Frame'] {
display: table;
- overflow: auto;
text-align: center;
border: 1px solid #c8ccd1;
- border-bottom: 0; // No border to caption
- border-collapse: collapse;
background-color: #f8f9fa;
+ width: 1px; // From https://stackoverflow.com/a/6536025
+ padding: 3px;
+
// Default to right alignment. This is needed since Parsoid only
specifies the
// alignment class when the alignment is explicitly set.
margin: 0.5em 0 1.3em 1.4em;
@@ -133,9 +115,22 @@
> img,
> video {
border: 1px solid #c8ccd1;
- margin: 3px;
background: #fff;
}
+ }
+
+ > figcaption {
+ display: block;
+
+ /* In mw-core the font-size is duplicated, 94% in thumbiner
+ * and again 94% in thumbcaption. 88.4% for font size of the
+ * caption results in the same behavior. */
+ font-size: 88.4%;
+ line-height: 1.4em;
+ text-align: left;
+
+ /* taken from .thumbcaption, plus .thumbinner */
+ padding: 3px;
}
}
@@ -144,17 +139,5 @@
> img,
> video {
border: 1px solid #eaecf0;
- }
-}
-
-/* Hide the caption for frameless and plain floated images */
-figure[typeof~='mw:Image/Frameless'],
-figure[typeof~='mw:Video/Frameless'],
-figure[typeof~='mw:Audio/Frameless'],
-figure[typeof~='mw:Image'],
-figure[typeof~='mw:Video'],
-figure[typeof~='mw:Audio'] {
- > figcaption {
- display: none;
}
}
--
To view, visit https://gerrit.wikimedia.org/r/366898
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2ff8e41d3cb973f56f6571d84fb662aed23b1f10
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Arlolra <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
