Re: [css-grid] Editorial: rename and move the placement algorithm

On Mon, May 5, 2014 at 1:58 PM, Simon Sapin <simon.sapin@exyr.org> wrote:
> I suggest:
>
> Section 6.2.1 "Automatic Grid Item Placement Algorithm" should be renamed to
> "Grid Item Placement Algorithm" (and similarly for <dfn>auto-placement
> algorithm</dfn>), since the algorithm takes also takes care of items that
> are not auto-positioned. It is important to define that this happens first,
> and that automatic placement only considers areas that are still "free"
> afterwards.
>
> This algorithm and its parent section 6.2 "Automatic Placement: the
> grid-auto-flow property" (and the corresponding introductory paragraph at
> the beginning of section 6) should be moved under section 8 "Placing Grid
> Items" (rather than section 6 "The Implicit Grid") since it is more strongly
> related to the former than the latter. Section 6 could say "The size of the
> <a>implicit grid</a> is determined by the <a>placement algorithm</a>"
> instead.

Moving/renaming the auto-placement algo into section 8 is fine.  But
section 6.2 (defining grid-auto-flow) seems like it should stay where
it is.  The current section 8 (Placing Grid Items) defines properties
that go on the grid items, but grid-auto-flow goes on the grid
*container*, along with the other grid-auto-* sizing properties.
Also, grid-auto-flow is settable alongside the other grid-auto-*
properties in the grid shorthand, but the grid placement properties
aren't.  It just feels like all of the grid-auto-* properties should
be grouped together if possible.

We've gone ahead and done the move/rename of the actual algorithm.

~TJ and fantasai

Received on Monday, 7 July 2014 19:48:40 UTC