[wp-trac] [WordPress Trac] #30770: Twenty Fifteen: Theme Review: Stylesheets and Scripts
WordPress Trac
noreply at wordpress.org
Fri Dec 19 16:00:27 UTC 2014
#30770: Twenty Fifteen: Theme Review: Stylesheets and Scripts
---------------------------+----------------------
Reporter: emiluzelac | Owner:
Type: defect (bug) | Status: closed
Priority: normal | Milestone:
Component: Bundled Theme | Version: 4.1
Severity: normal | Resolution: wontfix
Keywords: | Focuses:
---------------------------+----------------------
Comment (by philiparthurmoore):
Replying to [comment:8 emiluzelac]:
> Not really sure what is the best way, you can see:
https://themes.trac.wordpress.org/ticket/20079#comment:3 for example.
>
> This is not a question how to improve or do this better, guidelines
should apply to all themes equally and if it does not, it will came back
on us the reviewers :)
My apologies. I thought this was a code discussion, not a theme review
discussion. (Seriously, no sarcasm intended.) I have no idea what the
submission process for default themes is like so I cannot address that. My
points above were specifically about the code.
If this is causing an issue with review guidelines, why not change them?
What's wrong with adjusting the guidelines? Because it was done one way 4
months ago doesn't mean that it has to be done that way now. Jankiness and
flashing does absolutely occur with the JS detection happens too late;
shouldn't we be trying to address that performance issue rather than the
issue of whether or not a theme is in line with review guidelines?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/30770#comment:10>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list