[theme-reviewers] Second opinion on this ticket please

Tue Jun 4 10:04:56 UTC 2013

Thanks for your views guys :)
I've advised the developer that it is a no and referred them to the Theme Customizer API guidelines.

Date: Tue, 4 Jun 2013 14:08:24 +0530
From: abhik at itsabhik.com
To: theme-reviewers at lists.wordpress.org
Subject: Re: [theme-reviewers] Second opinion on this ticket please

Inline Styles are not allowed as long as I know (Chip mentioned it somewhere in my tickets as well as mailing list).

The Customizer CSS should be printed in <head> using wp_head() hook, not using inline styles.
Ref: https://codex.wordpress.org/Theme_Customization_API

*Offtopic
Emil, if you have time, can you please address this?
http://make.wordpress.org/themes/about/trac-ticket-request-queue/trac-ticket-request-queue-mar-2013/#comment-32644

Thanks

On Tue, Jun 4, 2013 at 1:25 PM, Emil Uzelac <emil at uzelac.me> wrote:

Inline styles, I tried to "fight" that, but I lost. :(

On Jun 4, 2013 2:37 AM, "Zulfikar Nore" <zulfikarnore at live.com> wrote:

Hello team,
Ticket: http://themes.trac.wordpress.org/ticket/12334#comment:5

Could I please have an admin's second opinion on this ticket on the issue of inline styles.I am of the opinion that this falls under the hard coded styling in the html document/templateand is therefore not be allowed.

What are your views on this?My suggestion to the developer would be to wrap those divs in need of inline styling in afunction and then reference that function in place of the div.

Thanks,Zulf 		 	   		  

_______________________________________________

theme-reviewers mailing list

theme-reviewers at lists.wordpress.org

http://lists.wordpress.org/mailman/listinfo/theme-reviewers

_______________________________________________

theme-reviewers mailing list

theme-reviewers at lists.wordpress.org

http://lists.wordpress.org/mailman/listinfo/theme-reviewers

_______________________________________________
theme-reviewers mailing list
theme-reviewers at lists.wordpress.org
http://lists.wordpress.org/mailman/listinfo/theme-reviewers 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130604/870a534e/attachment.html>