Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Errori di validazione HTML5 #96

Open
marcochiesi opened this issue Jun 19, 2015 · 7 comments
Labels
bug
Milestone

Comments

@marcochiesi
Copy link

@marcochiesi marcochiesi commented Jun 19, 2015

Ciao e grazie per aver creato questo plugin.

Vi segnalo che ho riscontrato 3 errori di validazione derivanti dal plugin con il validatore del W3C:

  1. Error: Element style not allowed as child of element body in this context. (Suppressing further errors from this subtree.)
    hoices --><style>.icc{m
  2. Error: Element style is missing required attribute scoped.
    <noscript><style>html{m
  3. Error: Stray end tag div.
    </a></div></div></nosc

Gli snippet sopra riportati sono troncati, ma si possono facilmente individuare come facenti parte di Italy Cookie Choices. Il terzo è dovuto ad un banale </div> di troppo che va tolto, mentre i primi due non sono di banale soluzione in quanto non è sufficiente aggiungere l'attributo scoped ma occorre rivedere un po' l'implementazione.

@overclokk

This comment has been minimized.

Copy link
Contributor

@overclokk overclokk commented Jun 21, 2015

Ciao @marcochiesi e grazie della notifica, faro dei test appena possibile.

@overclokk overclokk added this to the Version 2.2.x milestone Jun 26, 2015
@overclokk overclokk added the bug label Jun 26, 2015
@overclokk

This comment has been minimized.

Copy link
Contributor

@overclokk overclokk commented Jun 26, 2015

@marcochiesi il div di troppo è stato eliminato con la versione 2.2.0

Per quanto riguarda gli attributi del tag style sono opzionali https://developer.mozilla.org/it/docs/Web/HTML/Element/style

@marcochiesi

This comment has been minimized.

Copy link
Author

@marcochiesi marcochiesi commented Jun 26, 2015

Grazie per l'aggiornamento.

L'attributo scope del tag style è sì opzionale, tuttavia in caso non venga messo, è previsto che il tag si trovi in un contesto di metadata content.

Vedi anche la documentazione ufficiale del W3C relativa al tag style.

Si tratta di una cosa puramente formale, in quanto di fatto i browser interpretano correttamente lo stile. Il problema è che fa fallire la validazione con il validatore ufficiale del W3C, il che è abbastanza fastidioso.

@overclokk

This comment has been minimized.

Copy link
Contributor

@overclokk overclokk commented Jun 26, 2015

Inserito lo snippet in un tag div così ora il validatore non segnala più l'errore
Sarà disponibile nella prossima release.

@overclokk overclokk closed this Jun 26, 2015
@marcochiesi

This comment has been minimized.

Copy link
Author

@marcochiesi marcochiesi commented Jun 26, 2015

Grazie mille. Ottimo lavoro!

@overclokk

This comment has been minimized.

Copy link
Contributor

@overclokk overclokk commented Jun 26, 2015

👍

@overclokk overclokk reopened this Jul 1, 2015
@overclokk

This comment has been minimized.

Copy link
Contributor

@overclokk overclokk commented Jul 1, 2015

@marcochiesi devo riaprire la issue perché con la versione 2.2.2 ho dovuto momentaneamente togliere il fix al problema perché ne creava uno peggiore, incompatibilità con firefox e IE.

Se hai una soluzione migliore proponi pure, ad ogni modo vedo di trovare comunque un'alternativa.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.