column-gap is incorrectly marked as invalid if not applied to a grid container

VERIFIED FIXED in Firefox 69

Status

defect
P1
normal
VERIFIED FIXED
2 months ago
15 days ago

People

(Reporter: cers, Assigned: rcaliman)

Tracking

(Blocks 1 bug)

unspecified
Firefox 69
Dependency tree / graph

Firefox Tracking Flags

(firefox69 verified)

Details

Attachments

(2 attachments)

Posted file Test case

The property 'column-gap' is currently marked as invalid if not applied to a grid container, but this is false. It also applies to flex and multi-column containers.

STR:

  1. load attached test case
  2. inspect containers with class "gap"
  3. look for column-gap rule in Rules view

Thanks for filing!

Blocks: 1540753
No longer regressed by: 1306054
Assignee: nobody → rcaliman
Status: NEW → ASSIGNED
Priority: -- → P1

Splits the checks by appropriate container:

  • column-gap and gap for flex-container, grid-container and multi-column container
  • row-gap only for flex container and grid container

See CSS Box Alignment: https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Box_Alignment

Pushed by rcaliman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ed1862121e9a
Better checks for invalid CSS when using gap, column-gap and row-gap. r=miker,fluent-reviewers,flod
Status: ASSIGNED → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69

Confirmed issue with 69.0a1(2019-06-05).
Verified fix with 69.0a1 (2019-07-07).

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.