I do not know how to reply in the ticket. There is not option to reply there. <div><a href="http://themes.trac.wordpress.org/ticket/2478#comment:3">http://themes.trac.wordpress.org/ticket/2478#comment:3</a><br><br><div class="gmail_quote">
On Mon, Jan 10, 2011 at 2:15 PM, Philip M. Hofer (Frumph) <span dir="ltr"><<a href="mailto:philip@frumph.net">philip@frumph.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div bgcolor="#ffffff">
<div><font size="2" face="Arial">Read my comment again, I wrote that using @import
is alright.</font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial">This line:</font></div>
<div><font color="#008000" size="2"><font color="#008000" size="2">
<p><link rel="stylesheet" type="text/css" href="</p></font></font><font color="#0000ff" size="2"><font color="#0000ff" size="2"><?php</font></font><font size="2"> </font><font color="#0000ff" size="2"><font color="#0000ff" size="2">print</font></font><font size="2"> get_stylesheet_directory_uri();
</font><font color="#0000ff" size="2"><font color="#0000ff" size="2">?></font></font><font color="#008000" size="2"><font color="#008000" size="2">/css/ie.css" media="screen" /></font></font>
<p><font color="#008000" size="2"><font color="#008000" size="2"><font color="#000000" face="Arial">If an end user is using a child theme, it will try to find the
/css/ie.css in the child theme, instead of the main one in the parent theme,
whether you have a child theme or not, if an end user has one that is what will
happen.</font></font></font></p>
<p><font color="#008000" size="2"><font color="#008000" size="2"><font color="#000000" face="Arial">Change that to get_template_directory_uri() and check all of the
other references of get_stylesheet_directory you have to make sure they are as
they should be or need to switch to get_template_directory as
well.</font></font></font></p>
<p><font color="#008000" size="2"><font color="#008000" size="2"><font color="#000000" face="Arial">Looking over your previous tickets: <a href="http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-black-queen&order=priority" target="_blank">http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-black-queen&order=priority</a></font></font></font></p>
<p><font color="#008000" size="2"><font color="#008000" size="2"><font color="#000000" face="Arial">I am not seeing anyone say that there was no need, not a single
ticket there says it.</font></font></font></p></div>
<div><font size="2" face="Arial"></font> </div>
<blockquote style="border-left:#000000 2px solid;padding-left:5px;padding-right:0px;margin-left:5px;margin-right:0px"><div class="im">
<div style="font:10pt arial">----- Original Message ----- </div>
<div style="font:10pt arial;background:#e4e4e4"><b>From:</b>
<a title="fordmon@gmail.com" href="mailto:fordmon@gmail.com" target="_blank">Ford Mon</a> </div>
<div style="font:10pt arial"><b>To:</b> <a title="theme-reviewers@lists.wordpress.org" href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
</div>
</div><div><div></div><div class="h5"><div style="font:10pt arial"><b>Sent:</b> Monday, January 10, 2011 11:04
AM</div>
<div style="font:10pt arial"><b>Subject:</b> Re: [theme-reviewers] [WordPress
Themes] #2478: THEME: Black Queen- 1.3</div>
<div><br></div><span style="border-collapse:collapse;font-family:arial, sans-serif"> <b><font size="4">To Frumph</font></b></span><br><br>
<div>You wrote the requirement: - Stylesheets using @import are
alright, the wp_enqueue_style requirement</div> is if it's hardcoded
instead of enqueued, while inside of an -IF LTE IE<br> statement and such
is alright to hardcode as long as the proper<br> directory_uri is used.
<div><font color="#ff0000"><b>Why to
use wp_enqueue_style() if we have 1CSS ? Please
elaborate</b></font></div>
<div><font color="#ff0000"><br></font></div>
<div><font color="#ff0000"><span style="color:rgb(0,0,0)">Unfortunately
get_stylesheet_directory_uri(); is not it,
it's<br> get_template_directory_uri() which needs to be fixed before
being<br> accepted. If using get_stylesheet_directory_uri() and a
user is using a<br> child-theme it will check the css/ of that child
theme and not where it's<br> located in the main template.</span>
<b>Please explain what should I fix. I do not have a child
theme.</b> </font></div>
<div><font color="#ff0000"><br></font></div>
<div><font color="#ff0000"><br></font></div>
<div><font color="#ff0000"><font color="#000000">get_stylesheet_directory(_uri)</font><font color="#000000">() will check the childtheme first, if
no</font><br><font color="#000000"> childtheme it
will use the main parent theme.</font><br><font color="#000000"> get_template_directory(_uri)() will always check the
parent theme </font><b> The previous reviewer stated that there is no
need in get_stylesheet_directory(_uri) so I took it out. Guys, could you
come to the consensus<font> w</font>hat's wrong in
my theme? </b></font></div>
<div><font color="#ff0000"><br></font>
<div><br>
<div class="gmail_quote">On Sun, Jan 9, 2011 at 6:25 PM, WordPress Themes <span dir="ltr"><<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>></span>
wrote:<br>
<blockquote style="border-left:#ccc 1px solid;margin:0px 0px 0px 0.8ex;padding-left:1ex" class="gmail_quote">
<div>#2478: THEME: Black Queen -
1.3<br>------------------------------+-------------------------------<br> Reporter:
themesfromrobin | Owner:
joshstauffer<br></div> Type: theme
| Status:
closed<br>Resolution: not-approved |
Keywords:
theme-black-queen,<br>------------------------------+-------------------------------<br>Changes
(by Frumph):<br><br> * status: assigned => closed<br> *
resolution: => not-approved<br><br><br>Comment:<br><br> -
Stylesheets using @import are alright, the wp_enqueue_style
requirement<br> is if it's hardcoded instead of enqueued, while inside
of an -IF LTE IE<br> statement and such is alright to hardcode as long
as the proper<br> directory_uri is used.<br><br> Unfortunately
get_stylesheet_directory_uri(); is not it,
it's<br> get_template_directory_uri() which needs to be fixed before
being<br> accepted. If using get_stylesheet_directory_uri() and a
user is using a<br> child-theme it will check the css/ of that child
theme and not where it's<br> located in the main
template.<br><br> get_stylesheet_directory(_uri)() will check the
childtheme first, if no<br> childtheme it will use the main parent
theme.<br> get_template_directory(_uri)() will always check the parent
theme<br><br> Required's<br> => adjust all of the
get_stylesheet/template(_uri)() references to match<br> appropriate
locations to where the files are<br> => fix the missing px in
the #content .archive-meta<br><br> ^ Recommendations above
stand.<br><font color="#888888"><br>--<br>Ticket URL: <<a href="http://themes.trac.wordpress.org/ticket/2478#comment:3" target="_blank">http://themes.trac.wordpress.org/ticket/2478#comment:3</a>><br></font>
<div>
<div></div>
<div>WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><br>WordPress.org
Theme Directory Reviews<br></div></div></blockquote></div><br></div></div>
</div></div><p>
</p><hr>
<p></p>_______________________________________________<br>theme-reviewers
mailing
list<div class="im"><br><a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br></div><div class="im"><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</div><p></p></blockquote></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>