Closed
Bug 1318882
Opened 8 years ago
Closed 8 years ago
fit-content track sizing vs fit-content box size
Categories
(Developer Documentation Graveyard :: CSS, defect)
Developer Documentation Graveyard
CSS
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: MatsPalmgren_bugz, Assigned: sebo)
References
()
Details
https://developer.mozilla.org/en-US/docs/Web/CSS/fit-content
The description here is fine for using fit-content() as a track-size.
However, fit-content() is now also specified as a value for 'width' / 'height'
etc so I think this page should be moved to something more Grid specific, e.g.
https://developer.mozilla.org/en-US/docs/Web/CSS/Grid/fit-content
It might be good to add some text in the intro to disambiguate it
from this: https://drafts.csswg.org/css-sizing/#size-keywords
Alternatively, you could use this page to describe fit-content() in generic
terms and mention that it can be used for width/height etc as well as a track-size.
But please make sure to point out that we haven't implemented fit-content() for
width/height etc yet. That's bug 1312588.
Comment 1•8 years ago
|
||
Assigned to :sebo per bug 1281320 comment 12
Assignee: nobody → sebastianzartner
Assignee | ||
Updated•8 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•8 years ago
|
||
Thanks for filing this, Mats!
I've adjusted the page[1] to match both, the grid properties and the width and height related properties, as CSS properties are generally not placed under a spec. specific URL.
Mats, Jean-Yves, can you please do a quick review of the changes?
Sebastian
[1] https://developer.mozilla.org/en-US/docs/Web/CSS/fit-content$revision/1145732
Flags: needinfo?(mats)
Flags: needinfo?(jypenator)
Reporter | ||
Comment 3•8 years ago
|
||
In the opening statement, please change fit-content() to fit-content(argument),
otherwise it's a bit mysterious where that comes from in the later text.
(the added "argument" should probably be in italics like the other ones)
This is a bit vague: "The function can be used for different grid properties".
I would change that to: "The function can be used as a track size in CSS Grid properties"
Typo: in "It can also be used for width..." paragraph, there are two spaces in "the maximum"
In "Specifications": "Initial definition" should probably say "Defines the function
when used as a track size"
In "Browser compatibility", please change "Basic support" to "Supported as a track size
in CSS Grid properties" or some such.
Also, "Specifications" / "Browser compatibility" should probably list the two
types of use in the same order. And in general, perhaps it's better to start
with the 'width' usage since that applies generally to all boxes, whereas
track sizes are specific to Grid.
Flags: needinfo?(mats)
Assignee | ||
Comment 4•8 years ago
|
||
(In reply to Mats Palmgren (:mats) from comment #3)
> In the opening statement, please change fit-content() to
> fit-content(argument),
> otherwise it's a bit mysterious where that comes from in the later text.
> (the added "argument" should probably be in italics like the other ones)
All CSS function descriptions skip the arguments in the summary. See e.g. https://developer.mozilla.org/en-US/docs/Web/CSS/linear-gradient. The syntax definition and examples clarify the usage. The main reason is probably to keep the summary short and easy to read (especially for functions with more complex syntaxes). So I've kept it as is for consistency. Jean-Yves, I'd like to hear your opinion on this, though.
> This is a bit vague: "The function can be used for different grid
> properties".
> I would change that to: "The function can be used as a track size in CSS
> Grid properties"
Fixed.
> Typo: in "It can also be used for width..." paragraph, there are two spaces
> in "the maximum"
Fixed.
> In "Specifications": "Initial definition" should probably say "Defines the
> function when used as a track size"
I agree, it makes it clearer, so I've changed it. It's just that the first listed spec. normally always says "Initial definition". Jean-Yves, can you please confirm that it's ok to have the sentence there instead of "Initial definition"?
> In "Browser compatibility", please change "Basic support" to "Supported as a
> track size in CSS Grid properties" or some such.
"Basic support" as the first entry of the compatibility table is an even stronger unwritten rule than the above (actually it is part of a writing style guide example[1]). In consistence to the above, I've changed it, anyway. Though like above I'd like Jean-Yves to confirm whether that's ok.
> Also, "Specifications" / "Browser compatibility" should probably list the two
> types of use in the same order. And in general, perhaps it's better to start
> with the 'width' usage since that applies generally to all boxes, whereas
> track sizes are specific to Grid.
Agree, it's consistent with the above.
Sebastian
[1] https://developer.mozilla.org/en-US/docs/MDN/Contribute/Structures/Compatibility_tables#Browser_compatibility
Assignee | ||
Comment 5•8 years ago
|
||
I've updated the compatibility now.
Jean-Yves, I'm still waiting for your feedback regarding comment 4.
Sebastian
Assignee | ||
Comment 6•8 years ago
|
||
Opened these topics up for broader discussion at https://groups.google.com/forum/#!topic/mozilla.dev.mdc/r3GMufqMHV4.
Closing this bug now, as Mats' comments got fixed.
Sebastian
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(jypenator)
You need to log in
before you can comment on or make changes to this bug.
Description
•