Re: [review feedback] qa-css-charset

On 10/03/2014 14:25, Gunnar Bittersmann wrote:
> »»
> It is not enough to simply put @charset="utf-8" at the top of the page
> ««
>
> A stylesheet is not a page. Use ‘CSS file’ or ‘stylesheet’:
> It is not enough to simply put <code class="">@charset="utf-8"</code> at
> the top of the page CSS file

Fixed.

>
>
> »»
> The CSS3 Syntax specification
> ««
>
> Mark up as title:
> The <cite>CSS3 Syntax</cite> specification

Hmm. I did this then undid it, since CSS3 Syntax is not the actual title 
of the spec. So I'm using a short form, rather like saying "the HTML5 spec".

I also added a link.

>
>
> »»
> <h3><a style="" id="atcharset" name="atcharset" href="#atcharset">Using
> @charset</a></h3>
> ««
> »»
> <h3><a style="" id="http" name="http" href="#http">Using HTTP</a></h3>
> ««
>
> Remove style attribute.

Done

>
>
> »»
> any other<code class="kw" translate="no"></code> declaration
> ««
>
> Remove empty code element.

Done

>
>
> »»
> So for the time being you should rely on the <code class="kw"
> translate="no">@charset</code> or HTTP header declarations instead. This
> has the additional advantage that it helps people looking at the source
> code to ascertain what the encoding of the page is.
> ««
>
> This refers to @charset, but not to the HTTP header. It might read like
> this:
> The former has the additional advantage …

Done.
>
>
> Cheers,
> Gunnar
>
>

Received on Monday, 10 March 2014 18:10:49 UTC