[uf-discuss] How's my hReview?

[Brian Suda]

It is not a valid review yet, you are missing a few things. The
following is a snippit of the site.

<div class="hreview">
<h1>West Midland Bird Club</h1>
<h2 id="content" class="item">'Warwickshire Birding 2005', by Steve Seal</h2>
<p class="center">DVD<br>&pound;5.99</p>

<div class="pic-right"><img class="photo"
src="../images/biblio/WarksBirding2005.jpg" width="250" height="455"
alt="" title="cover of Warwickshire Birding 2005"></div>

<div class="description">...</div>
</div>

You correctly have the class="hreview", nested inside that there is a
class="item" (you'd be suprised how many reviews don't actualy have an
item being reviewed!)

The problem is that there is no class="fn" nested in the class="item".
There are a few ways to fix this.
A) just add an FN inside the H2.
<h2 id="content" class="item"><span class="fn">'Warwickshire Birding
2005', by Steve Seal</span></h2>

B) create an "outer div container". You have a class="photo" which is
another hReview property - assuming you are actually intending that to
be part of the hReview, that needs to be nested inside the
class="item" as well.

<div class="hreview">
  <div class="item">
    <h1>West Midland Bird Club</h1>
    <h2 id="content" class="fn">'Warwickshire Birding 2005', by Steve Seal</h2>
    <p class="center">DVD<br>&pound;5.99</p>
    <div class="pic-right"><img class="photo"
src="../images/biblio/WarksBirding2005.jpg" width="250" height="455"
alt="" title="cover of Warwickshire Birding 2005"></div>
  </div> <!-- ends item -->
<div class="description">...</div>
</div>

There is no RATING, which is OK that is optional, but IMHO that is
kinda the point of a review.

finnaly, the reviewer hCard is also incorrect:
<p id="credit"><span class="reviewer vcard fn">Andy Mabbett</span><br>
<span class="dtreviewed" title="200603">March 2006</span></p>

you need to nest the class="fn" inside the class="vcard"
<p id="credit" class="reviewer vcard"><span class="fn">Andy Mabbett</span><br>
<span class="dtreviewed" title="200603">March 2006</span></p>

i hope this helps. If you make those changes let us know and we can
take another pass at it and see if there are any other errors.

-brian

On 8/29/06, Andy Mabbett <andy at pigsonthewing.org.uk> wrote:
>
> Here's my first stab at marking up one of my reviews with hReview:
>
>         http://www.westmidlandbirdclub.com/reviews/WarksBirding2005.htm
>
> how does it look?
>
> --
> Andy Mabbett
>                 Say "NO!" to compulsory ID Cards:  <http://www.no2id.net/>
>
>                 Free Our Data:  <http://www.freeourdata.org.uk>
> _______________________________________________
> microformats-discuss mailing list
> microformats-discuss at microformats.org
> http://microformats.org/mailman/listinfo/microformats-discuss
>


-- 
brian suda
http://suda.co.uk