Opened 8 months ago
Closed 7 months ago
#9569 closed theme (not-approved)
THEME: Newschannel - 1.0
| Reported by: | citizenjournal | Owned by: | Frank Klein |
|---|---|---|---|
| Priority: | major | Keywords: | theme-newschannel |
| Cc: | citizenjournal@… |
Description
Newschannel - 1.0
Newschannel is a pretty versatile theme, that places the emphasis of the home page and archive pages on featured images. It would be great for a personal blog, but would also be good for a local news site or even a photography blog.
Theme URL - http://citizenjournal.net/newschannel/
Author URL - http://citizenjournal.net/
SVN - http://themes.svn.wordpress.org/newschannel/1.0
ZIP - http://wordpress.org/extend/themes/download/newschannel.1.0.zip?nostats=1
All previous tickets for this theme: http://themes.trac.wordpress.org/query?keywords=~theme-newschannel
Change History (3)
comment:1 Changed 7 months ago by emiluzelac
- Owner set to Frank Klein
- Status changed from new to assigned
comment:2 Changed 7 months ago by Frank Klein
comment:3 Changed 7 months ago by Frank Klein
- Resolution set to not-approved
- Status changed from assigned to closed
So here is my theme review: the theme respects all the Theme Review Guidelines except that requiring valid HTML.
The pages don't pass the W3C Validator, due to what I suspect are issues with the search form. The first issue is that the form is placed in the hgroup in header.php, which is not semantically correct. The second issue is that there are two search forms on the pages, so there are duplicate ID's in the HTML. There is also an issue of incorrectly closed DIVs on the page template.
So unfortunately I cannot approve the theme at this point, but in my opinion some rather quick fixes to the above mentioned issues are all that is needed for the theme approved.
Please don't hesitate to comment in the ticket if you have any questions concerning the review.


Hi,
just a quick note to let you know that I'm working on the review, I'll finish it this Thursday.
Thanks for your patience,
Fränk