Closed
Bug 1856524
Opened 8 months ago
Closed 6 months ago
[css-properties-values-api] Use canonical units for custom properties with font-relative values
Categories
(Core :: CSS Parsing and Computation, enhancement)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
122 Branch
Tracking | Status | |
---|---|---|
firefox122 | --- | fixed |
People
(Reporter: zrhoffman, Assigned: zrhoffman)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
This will require passing the computed::Context's StyleBuilder all the way to properties_and_values::ComputedValue::compute.
Updated•8 months ago
|
Blocks: css-properties-values-api
Assignee | ||
Comment 1•6 months ago
|
||
In order to call Cascade::apply_prioritary_properties
while the computed::Context
is borrowed from the CustomPropertiesBuilder
, the computed::Context
will probably need to be passed as an argument instead of the Cascade
having a context
field.
Assignee | ||
Comment 2•6 months ago
|
||
Now that registered custom properties containing font-relative units are
calculated accurately, the two syntaxes tending to use font-relative
units the most, <length> and <length-percentage>, can be computed.
Depends on D194023
Updated•6 months ago
|
Assignee: nobody → zach
Status: NEW → ASSIGNED
Updated•6 months ago
|
Attachment #9364383 -
Attachment description: Bug 1856524 - Compute <length> and <length-percentage>. r=zsun,fredw,#style → Bug 1856524 - [css-properties-values-api] Compute <length> and <length-percentage>. r=zsun,fredw,#style
Assignee | ||
Comment 3•6 months ago
|
||
Unassigning since D194024 is not landing yet.
Assignee: zach → nobody
Status: ASSIGNED → NEW
Updated•6 months ago
|
Assignee: nobody → zach
Status: NEW → ASSIGNED
Pushed by zach@zrhoffman.net: https://hg.mozilla.org/integration/autoland/rev/a35aa51e1bfc [css-properties-values-api] Compute <length> and <length-percentage>. r=firefox-style-system-reviewers,emilio
Comment 5•6 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
status-firefox122:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•