- From: csswg-test-issue-mover <web-platform-tests-notifications@w3.org>
- Date: Fri, 31 Mar 2017 13:04:39 GMT
- To: public-web-platform-tests-notifications@w3.org
Originally posted as https://github.com/w3c/csswg-test/pull/1057#issuecomment-193508236 by @dholbert on 07 Mar 2016, 23:37 UTC:
> Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions.
>
> ---
>
> _[css-flexbox-1/definite-cross-sizes.html, line 38 [r1]](https://reviewable.io:443/reviews/w3c/csswg-test/1057#-KCIQ8v-XjhJ4_jhcnHC:-KCIQ8v-XjhJ4_jhcnHD:-1767485284) ([raw file](https://github.com/w3c/csswg-test/blob/9dbf389412cbc8299de96a1a59ea4723b226226a/css-flexbox-1/definite-cross-sizes.html#L38)):_
> Nit: This sentence ("Tests that we ...") ends in a period, but all the ones after it do not.
>
> Really, they might all want to end in a colon, since they're all referring to the content directly after them...?
>
> Regardless, probably best to make these consistent.
>
> ---
>
> _[css-flexbox-1/definite-cross-sizes.html, line 65 [r1]](https://reviewable.io:443/reviews/w3c/csswg-test/1057#-KCIQTB63ZtqoLvw1u9z:-KCIQTB63ZtqoLvw1uA-:400756160) ([raw file](https://github.com/w3c/csswg-test/blob/9dbf389412cbc8299de96a1a59ea4723b226226a/css-flexbox-1/definite-cross-sizes.html#L65)):_
> Nit: s/an definite/a definite/ (drop the "n" from "an")
>
> ---
>
> _Comments from the [review on Reviewable.io](https://reviewable.io:443/reviews/w3c/csswg-test/1057)_
>
> <!-- Sent from Reviewable.io -->
View on GitHub: https://github.com/w3c/web-platform-tests/pull/5282#issuecomment-290706355
Received on Friday, 31 March 2017 13:04:52 UTC