mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/Popups
synced 2024-11-28 01:10:04 +00:00
Resize popup to 403px
Restores the max-size introduced in I6036968a2, after conflicting changes to the popup layout. Bug: T246029 Change-Id: I82ea48929b44d97d78112bd96ad75724ae6f46af
This commit is contained in:
parent
789cedc168
commit
50c0e46b0a
|
@ -67,7 +67,7 @@
|
||||||
"bundlesize": [
|
"bundlesize": [
|
||||||
{
|
{
|
||||||
"path": "resources/dist/index.js",
|
"path": "resources/dist/index.js",
|
||||||
"maxSize": "13.82kB"
|
"maxSize": "13.6kB"
|
||||||
}
|
}
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
|
|
|
@ -28,12 +28,11 @@
|
||||||
|
|
||||||
.mwe-popups-extract {
|
.mwe-popups-extract {
|
||||||
margin-right: 0;
|
margin-right: 0;
|
||||||
margin-bottom: @lineHeight;
|
margin-bottom: @popupPadding;
|
||||||
max-height: inherit;
|
max-height: inherit;
|
||||||
|
|
||||||
.mwe-popups-scroll {
|
.mwe-popups-scroll {
|
||||||
// 8px caused by different margin-bottom, 55px here vs. 47px upstream
|
max-height: 403px - 2 * @popupPadding;
|
||||||
max-height: 200px - 8px + 7 * @lineHeight;
|
|
||||||
overflow: auto;
|
overflow: auto;
|
||||||
padding-right: @popupPadding;
|
padding-right: @popupPadding;
|
||||||
}
|
}
|
||||||
|
|
|
@ -112,8 +112,8 @@ module.exports = ( env, argv ) => ( {
|
||||||
// Minified uncompressed size limits for chunks / assets and entrypoints. Keep these numbers
|
// 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
|
// 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.
|
// well understood. Related to bundlesize minified, gzipped compressed file size tests.
|
||||||
maxAssetSize: 42.6 * 1024,
|
maxAssetSize: 41.4 * 1024,
|
||||||
maxEntrypointSize: 42.6 * 1024,
|
maxEntrypointSize: 41.4 * 1024,
|
||||||
|
|
||||||
// The default filter excludes map files but we rename ours.
|
// The default filter excludes map files but we rename ours.
|
||||||
assetFilter: ( filename ) => !filename.endsWith( srcMapExt )
|
assetFilter: ( filename ) => !filename.endsWith( srcMapExt )
|
||||||
|
|
Loading…
Reference in a new issue