From b3f1fb531314cfb1759f737dbf63222b95973ea1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bartosz=20Dziewo=C5=84ski?= Date: Fri, 21 Jun 2024 23:32:29 +0200 Subject: [PATCH] Definition list has no left margin, only definition items do This matches how the indents are applied by other skins. Bug: T367092 Change-Id: I7cf78065bd8ddd6c97a242dfcce586704029ccc1 --- .../skins.minerva.base.styles/content/lists.less | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/resources/skins.minerva.base.styles/content/lists.less b/resources/skins.minerva.base.styles/content/lists.less index 0a5355109..bbf232f0f 100644 --- a/resources/skins.minerva.base.styles/content/lists.less +++ b/resources/skins.minerva.base.styles/content/lists.less @@ -36,21 +36,16 @@ /* stylelint-enable no-descending-specificity */ } -/* As in Vector, indenting of definition lists, and bolding of definition terms is generically useful */ dl { - margin: 0.2em 0 0.5em 1em; + /* Unlike other lists, a definition list has no left margin, only definition items do */ + margin: 0.2em 0 0.5em 0; + /* As in Vector, indenting of definition items, and bolding of definition terms is generically useful */ dt { font-weight: bold; } - /* - * Historically Minerva used a CSS reset to reset the margin of dd elements to 0. - * The main use case here is talk pages which can have significant intendation levels. - * This should likely be reconsidered for appropriate semantic use of dd elements and - * scoped to talk pages at a later date. - */ dd { - margin: 0; + margin: 0 0 0 1em; } }