[theme-reviewers] Theme Problems

[Edward Caissie]

Item 1. should probably be addressed but is not a huge issue.

Item 2. should be OK although there is no context without at least the trac
ticket for reference.

Item 3. can be a bit tricky (at least I have found it to be when this was
first introduced into the test data), first are you certain your test data
contains the proper HTML markup to emphasize and bold the words
respectively; and, second, the general aesthetic style of the post title
may be more appropriate than a marked-up version would be ... it's a "very
nice to do" item which the theme author should be aware of but IMO a very
niche use-case that should never cause a theme to be resolved as
not-approved.

Over-all, again without the context of the trac ticket, I cannot say one
way or the other if I would approve or not-approve a theme with these
issues.


- Cais.

Edward Caissie
aka Cais.


On Wed, Oct 2, 2013 at 8:01 PM, Phillip Wells <phillipwells at hotmail.co.nz>wrote:

> I am currently reviewing a theme that:
> 1. fails the no post title test as no post title is displayed and there
> is no way of linking to the post
>
> 2. Repeating sticky posts later on in blog post list on front page.
>
>
> 3. Failed title with markup test as the text "with" was not indented and
> the text "markup" was not bold
>
>
> Should i reject the theme due to them or jst alert the person to these
> problems as they are minor and there are no other problems?
>
>
> _______________________________________________
> 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/20131002/e3be4e13/attachment.html>