Add (i) info icon to collapsible replacement message

This avoids pulling in the entirety of OOjs, with the disadvantage
that we have to copy a little bit of CSS. I copied parts of this
patch from I2a28666.

There might be a better way to do this, with less code. E.g. is
there a better way to construct these HTML elements?

Bug: T220208
Change-Id: I024155f3ff0f57de1d68bbaf37bfb9e81e692bd0
This commit is contained in:
Thiemo Kreuz 2020-10-28 17:57:43 +01:00 committed by Adam Wight
parent 468ca01266
commit f4d696e6bf
6 changed files with 41 additions and 16 deletions

View file

@ -112,6 +112,12 @@
}
},
"ResourceModules": {
"ext.popups.icons": {
"class": "ResourceLoaderOOUIIconPackModule",
"icons": [
"infoFilled"
]
},
"ext.popups.images": {
"selector": ".mw-ui-icon-{name}:before",
"class": "ResourceLoaderImageModule",
@ -202,6 +208,7 @@
"popups-refpreview-jump-to-reference"
],
"dependencies": [
"ext.popups.icons",
"ext.popups.images",
"mediawiki.experiments",
"mediawiki.jqueryMsg",

Binary file not shown.

Binary file not shown.

View file

@ -67,8 +67,15 @@ export function renderReferencePreview(
// We assume elements that benefit from being collapsible are to large for the popup
$el.find( '.mw-collapsible' ).replaceWith( () => $( '<div>' )
.addClass( 'mwe-collapsible-placeholder' )
.append(
$( '<span>' )
.addClass( 'mw-ui-icon mw-ui-icon-element mw-ui-icon-infoFilled' ),
$( '<div>' )
.addClass( 'mwe-collapsible-placeholder-label' )
.text( mw.msg( 'popups-refpreview-collapsible-placeholder' ) )
.addClass( 'mwe-collapsible-placeholder' ) );
)
);
// Undo remaining effects from the jquery.tablesorter.js plugin
$el.find( 'table.sortable' ).removeClass( 'sortable jquery-tablesorter' )

View file

@ -6,6 +6,7 @@
.mwe-popups.mwe-popups-type-reference {
.mwe-popups-title {
margin: 0 0 @popupPadding;
}
.mw-ui-icon {
vertical-align: middle;
@ -24,7 +25,6 @@
margin-left: -2px;
}
}
}
.mwe-popups-extract {
margin-right: 0;
@ -63,6 +63,17 @@
.mwe-collapsible-placeholder {
font-weight: bold;
margin: 1em 0;
position: relative;
// This mirrors .oo-ui-iconElement-icon
.mw-ui-icon {
position: absolute;
}
// This mirrors .oo-ui-messageWidget > .oo-ui-labelElement-label
.mwe-collapsible-placeholder-label {
margin-left: 2em;
}
}
}

View file

@ -112,8 +112,8 @@ module.exports = ( env, argv ) => ( {
// Minified uncompressed size limits for chunks / assets and entrypoints. Keep these numbers
// up-to-date and rounded to the nearest 10th of a kibibyte so that code sizing costs are
// well understood. Related to bundlesize minified, gzipped compressed file size tests.
maxAssetSize: 42.3 * 1024,
maxEntrypointSize: 42.3 * 1024,
maxAssetSize: 42.5 * 1024,
maxEntrypointSize: 42.5 * 1024,
// The default filter excludes map files but we rename ours.
assetFilter: ( filename ) => !filename.endsWith( srcMapExt )