Re: [css-grid] Typos in section 6.2.1 (Auto Placement Algorithm)

On 04/23/2014 11:27 AM, Daniel Holbert wrote:
> Hi folks,
>
> Writing to report 3 small issues that I ran across this morning in the
> Grid ED, in Section 6.2.1 (Auto Placement Algorithm).
>
> Spec link: http://dev.w3.org/csswg/css-grid/#auto-placement
>
> (1) The following disclaimer needs to mention "block" & "inline" (whose
> mentions need to be swapped whenever row & column are swapped):
>
>    # this algorithm is written with the assumption that
>    # grid-auto-flow is set to row. If it is instead set
>    # to column, swap all mentions of rows and columns
>    # in this algorithm.
>
> The algorithm has several mentions of block & inline start & end edges
> (e.g. "Set the column position of the cursor to be equal to the
> inline-start index of the grid item").  These usages become incorrect if
> we swap the "row/column" terms without also swapping block/inline terms.

Added block & inline to the swapping, but also, switched to using
row-start/etc. so those terms don't even appear anymore.

> (2) Typo in Step 4:
>    # Initially the auto-placement cursor at is specified
>
> s/at is/is/.

Fixed.

> (3) Typo later in Step 4:
>    # column and row position, respectively Otherwise,
>
> This needs a period after "respectively".

Couldn't find this error. Likely disappeared in a round of edits.

~fantasai

Received on Friday, 19 December 2014 01:37:52 UTC