Use variables for padding content

On the long term we want to provide access to these variables in other
extensions. e.g. VisualEditor

Bug: T259331
Change-Id: Ibbc6f1905ea384a3d159088e3f5eca947eb6ec08
This commit is contained in:
jdlrobson 2020-09-17 11:52:01 -07:00 committed by Jdlrobson
parent da26e09bca
commit 839876ba82
2 changed files with 6 additions and 1 deletions

View file

@ -1,5 +1,6 @@
@import '../../variables.less'; @import '../../variables.less';
@import 'mediawiki.mixins.less'; @import 'mediawiki.mixins.less';
@import 'mediawiki.skin.variables.less';
// Putting a `skin-vector-max-width` class on the body and wrapping the rules // Putting a `skin-vector-max-width` class on the body and wrapping the rules
// herein enables the ability to feature flag the max-width design. This is // herein enables the ability to feature flag the max-width design. This is
@ -16,7 +17,9 @@
// TODO: Remove @padding-content in variables.less when the max-width layout // TODO: Remove @padding-content in variables.less when the max-width layout
// becomes the default layout since the value is different between legacy and // becomes the default layout since the value is different between legacy and
// latest. // latest.
@padding-content: 1.25em 0.5em 1.5em 0.5em; // Note this uses variables defined in mediawiki.skin.variables so that VisualEditor can read them
// see T259331.
@padding-content: @padding-top-content @padding-horizontal-content 1.5em;
// We want ~60px of space between the end of the sidebar and the start of the // We want ~60px of space between the end of the sidebar and the start of the
// content container for aesthetic reasons. The sidebar is already displaced // content container for aesthetic reasons. The sidebar is already displaced
// -30px so we simply add 30px of space to the width of the sidebar. // -30px so we simply add 30px of space to the width of the sidebar.

View file

@ -64,6 +64,8 @@
// Use `rgba()` notation for better Safari support, see T254489. // Use `rgba()` notation for better Safari support, see T254489.
@border-color-portal-heading-transparent: rgba( red( @border-color-portal-heading ), green( @border-color-portal-heading ), blue( @border-color-portal-heading ), 0 ); @border-color-portal-heading-transparent: rgba( red( @border-color-portal-heading ), green( @border-color-portal-heading ), blue( @border-color-portal-heading ), 0 );
@padding-content: 1em; @padding-content: 1em;
@padding-top-content: 1.25em;
@padding-horizontal-content: 0.5em;
// Navigation // Navigation
@background-color-secondary: #f6f6f6; @background-color-secondary: #f6f6f6;