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 <abrea...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits