Skip to content

Commit 8858ac6

Browse files
authored
fix(css): Remove ::first-line from margin* props (#706)
<!-- 🙌 Thanks for contributing! Adding details below will help us to merge your PR faster. --> <!-- Commits need to adhere to conventional commits and only `fix:` and `feat:` commits are added to the release notes. --> <!-- https://www.conventionalcommits.org/en/v1.0.0/#examples --> ### Description Correct the `alsoAppliesTo` for the CSS `margin` and `margin-{top,bottom,left,right.trim}` properties. ### Motivation The data claims that `margin*` properties apply to the `::first-line` pseudoelement, which is untrue. They apply to `::first-letter`, but not `::first-line`. As reported by @Akindin in mdn/content#31806. ### Additional details <!-- 🔗 Link to documentation, bug trackers, source control, or other places providing more context --> ### Related issues and pull requests Fixes mdn/content#31806
1 parent 4f43ef9 commit 8858ac6

1 file changed

Lines changed: 6 additions & 12 deletions

File tree

css/properties.json

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6384,8 +6384,7 @@
63846384
],
63856385
"order": "uniqueOrder",
63866386
"alsoAppliesTo": [
6387-
"::first-letter",
6388-
"::first-line"
6387+
"::first-letter"
63896388
],
63906389
"status": "standard",
63916390
"mdn_url": "https://developer.mozilla.org/docs/Web/CSS/margin"
@@ -6458,8 +6457,7 @@
64586457
"computed": "percentageAsSpecifiedOrAbsoluteLength",
64596458
"order": "uniqueOrder",
64606459
"alsoAppliesTo": [
6461-
"::first-letter",
6462-
"::first-line"
6460+
"::first-letter"
64636461
],
64646462
"status": "standard",
64656463
"mdn_url": "https://developer.mozilla.org/docs/Web/CSS/margin-bottom"
@@ -6532,8 +6530,7 @@
65326530
"computed": "percentageAsSpecifiedOrAbsoluteLength",
65336531
"order": "uniqueOrder",
65346532
"alsoAppliesTo": [
6535-
"::first-letter",
6536-
"::first-line"
6533+
"::first-letter"
65376534
],
65386535
"status": "standard",
65396536
"mdn_url": "https://developer.mozilla.org/docs/Web/CSS/margin-left"
@@ -6552,8 +6549,7 @@
65526549
"computed": "percentageAsSpecifiedOrAbsoluteLength",
65536550
"order": "uniqueOrder",
65546551
"alsoAppliesTo": [
6555-
"::first-letter",
6556-
"::first-line"
6552+
"::first-letter"
65576553
],
65586554
"status": "standard",
65596555
"mdn_url": "https://developer.mozilla.org/docs/Web/CSS/margin-right"
@@ -6572,8 +6568,7 @@
65726568
"computed": "percentageAsSpecifiedOrAbsoluteLength",
65736569
"order": "uniqueOrder",
65746570
"alsoAppliesTo": [
6575-
"::first-letter",
6576-
"::first-line"
6571+
"::first-letter"
65776572
],
65786573
"status": "standard",
65796574
"mdn_url": "https://developer.mozilla.org/docs/Web/CSS/margin-top"
@@ -6592,8 +6587,7 @@
65926587
"computed": "asSpecified",
65936588
"order": "perGrammar",
65946589
"alsoAppliesTo": [
6595-
"::first-letter",
6596-
"::first-line"
6590+
"::first-letter"
65976591
],
65986592
"status": "experimental",
65996593
"mdn_url": "https://developer.mozilla.org/docs/Web/CSS/margin-trim"

0 commit comments

Comments
 (0)