hreview-creator-feedback

From Microformats

hReview creator feedback

This page is for collecting bugs, suggested fixes, and feature requests for the hReview creator (http://microformats.org/code/hreview/creator)

Please list your name, and indicate which version (even just a date and time helps) of the creator that you are providing feedback on. Thanks!


Table of contents

Bug Reports

  • word "date*" in "review date*" jumps on the next row and moves "reviewer*" down in Safari (var. 2.0.4 (419.3)). It looks like "review" is label for date, "date*" is label for text field and "reviewer*" is label for empty row. The same bug in the latest WebKit. reported by Dmitry Baranovskiy at 2006-07-20 (link to mail archives?)
    • I can't currently reproduce this. (see email (http://microformats.org/discuss/mail/microformats-discuss/2006-July/004913.html)) --RyanKing 15:00, 24 Jul 2006 (PDT)
  • I didn't noticed value of url field in generated code. reported by Dmitry Baranovskiy at 2006-07-20 (link to mail archives?)
    • this needs clarification (see email (http://microformats.org/discuss/mail/microformats-discuss/2006-July/004914.html)) --RyanKing 15:02, 24 Jul 2006 (PDT)
  • JavaScript error in Firefox 2.0.0.11 in XP on page load --TravisHardiman 08:16, 4 Feb 2008 (PST)
Error: pad is not defined
Source File: http://microformats.org/code/hreview/creator
Line: 230
      $('#review-date-day').val(pad(d.getDate()));
  • Also JavaScript error when page is submitted --TravisHardiman 08:16, 4 Feb 2008 (PST)
Error: format_dt is not defined
Source File: http://microformats.org/code/hreview/creator
Line: 92
      var dt = format_dt(review_date_year, review_date_month, review_date_day, review_date_hour, review_date_minute, timezone);

Suggested Tweaks and Fixes

  • Should "photo" be inside an "item" as well as "fn" and "url"? reported by Dmitry Baranovskiy at 2006-07-20 (link to mail archives?)

Feature Requests

  • ...


Related Pages

Personal tools