[theme-reviewers] Fwd: THEME: Rachel - 0.8 (closed theme: not-accepted)

Edward Caissie edward.caissie at gmail.com
Sun Jun 27 02:29:39 UTC 2010


A quick fix is to put the wp_footer() call inside the "container" div. It's
currently outside.

Just a suggestion ...


Cais.

On Sat, Jun 26, 2010 at 10:09 PM, // ravi <ravi at g8o.net> wrote:

> On Jun 26, 2010, at 10:07 PM, Tom Lany wrote:
> > Comment from Cais (another theme reviewer/admin):
> >
> > I really like the theme but you might look at how it handles items hooked
> into the footer.
> >
> > I use one of my simpler plugins (BNS Login) that hooks into the footer
> and find it firmly positioned at the top of the page!?
> >
> > I would expect the footer to be at the bottom of the page ...
> >
> > Cais.
> >
> > If you could take a look at this, that would be great.
> >
>
> Tom, will do. Installing the plugin right now, on my development blog.
>
>        —ravi
>
>
> > Tom Lany
> > http://tomlany.net
> >
> >
> > On 6/26/10 9:01 PM, // ravi wrote:
> >> On Jun 26, 2010, at 9:16 PM, Tom Lany wrote:
> >>
> >>> I am recommending that an admin look at this and that it be approved.
> See feedback: http://themes.trac.wordpress.org/ticket/176.  Great work!
> >>>
> >>>
> >> Tom,
> >>
> >> thank you and I must once again say that your feedback was excellent!
> >>
> >>      —ravi
> >>
> >>
> >>
> >>> Tom Lany
> >>> http://tomlany.net
> >>>
> >>>
> >>>
> >>>
> >>> On 6/26/10 6:03 PM, // ravi wrote:
> >>>
> >>>> On Jun 26, 2010, at 6:27 PM, Tom Lany wrote:
> >>>>
> >>>>
> >>>>> Thanks for the note.  Feedback ha been posted:
> http://themes.trac.wordpress.org/ticket/173
> >>>>>
> >>>>>
> >>>>>
> >>>> Ah, good catch on the function_exists() naked literal. That’s fixed as
> well as a few other similar ones and also the content tag. Uploaded 0.92.
> Sorry for the confusion regarding 0.95. 0.91 was the previous number and
> this one is 0.92.
> >>>>
> >>>> thank you,
> >>>>
> >>>>    —ravi
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>> Tom Lany
> >>>>> http://tomlany.net
> >>>>>
> >>>>>
> >>>>> On 6/26/10 12:34 PM, // ravi wrote:
> >>>>>
> >>>>>
> >>>>>> On Jun 25, 2010, at 5:56 PM, Tom Lany wrote:
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>> Thanks for fixing these issues.  The theme looks a lot better.
>  Take a look here: http://themes.trac.wordpress.org/ticket/166
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>> Hello Tom,
> >>>>>>
> >>>>>> Fixed all of the above (I hope!), added license.txt and background
> colour/image support. Uploaded 0.95. Have at it! ;-)
> >>>>>>
> >>>>>>  —ravi
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>> Thanks!
> >>>>>>>
> >>>>>>> Tom Lany
> >>>>>>> http://tomlany.net
> >>>>>>>
> >>>>>>> On 6/25/10 4:04 PM, // ravi wrote:
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>> On Jun 22, 2010, at 11:16 PM, Tom Lany wrote:
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>> Thanks for the note.  I was the person who reviewed this theme.
>  If you want to leave the code for the pages file alone, that is probably
> fine.  I just suggested this as it is what is commonly done in most
> WordPress themes.
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>> Hello Tom,
> >>>>>>>>
> >>>>>>>> thanks to you, then, for your detailed review notes. I have fixed
> (to my knowledge) all the issues identified and posted 0.9 of the theme for
> review (a minute ago). I did leave the code for the pages file as is, but
> added a check to the common code to not display date stamp for pages (as
> suggested in the Codex).
> >>>>>>>>
> >>>>>>>> Additionally I have passed the theme XHTML and CSS through the W3C
> validator and cleaned it up so the only warnings/errors that remain are the
> -moz, -webkit shadow and border radius stuff. FYI, since my development blog
> is not visible outside, I used one of my external blogs (http://0sum.org/).
> Various other things have been fixed as well: esc_attr() for relevant PHP
> generated HTML attributes, IE specific CSS, use of register_sidebar_widget()
> was removed, a few hints for added for the collapsibility of sidebar
> widgets, meta tags were made single-line, capitalised stylesheet elements
> were made lowercase, the PHP errors/warnings noted in the Trac ticket were
> resolved, reference to ‘global $_POST’ was removed, an icon that was not GPL
> (from pixel-mixer, as you correctly noted) was replaced, 404 page title was
> set to standard title size.
> >>>>>>>>
> >>>>>>>> I also made sure that the search widget works (please let me know
> if there is some use case that I am missing). I have not yet created a print
> style for this theme. That will be significant work, to be done right, and I
> will work on that shortly.
> >>>>>>>>
> >>>>>>>> Regards,
> >>>>>>>>
> >>>>>>>>        —ravi
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>> Thanks for your submission!
> >>>>>>>>>
> >>>>>>>>> Tom Lany
> >>>>>>>>> http://tomlany.net
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>> ---------- Forwarded message ----------
> >>>>>>>>>> From: // ravi<ravi at g8o.net>
> >>>>>>>>>> Date: Tue, Jun 22, 2010 at 10:58 PM
> >>>>>>>>>> Subject: Re: THEME: Rachel - 0.8 (closed theme: not-accepted)
> >>>>>>>>>> To: Edward Caissie<edward.caissie at gmail.com>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>> Great feedback, Edward. I will work on fixing the validation
> etc. Regarding icons, those links were copied over from an older version and
> I have stripped most icons out of the theme. I will double check and update
> the links or replace the icons. The one place I disagree is regarding
> copying code from single.php. I would rather not have duplicated code.
> >>>>>>>>>>
> >>>>>>>>>> Regards,  --ravi
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>> On Jun 22, 2010, at 10:52 PM, Edward Caissie<
> edward.caissie at gmail.com>     wrote:
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>> Please see the details on this ticket in Trac:
> http://themes.trac.wordpress.org/ticket/8
> >>>>>>>>>>>
> >>>>>>>>>>> Thank you,
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>> The WordPress.org Theme Team
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>
> >>>>>>
> >>>>
> >>>>
> >>
>
> _______________________________________________
> 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/mailman/private/theme-reviewers/attachments/20100626/981930bc/attachment.htm>


More information about the theme-reviewers mailing list