hreview-creator-issues: Difference between revisions

From Microformats Wiki
Jump to navigation Jump to search
(rolpaslirel)
m (Reverted edits by AllirIcpas (Talk) to last version by Tantek)
Line 1: Line 1:
boliricmo
<h1>hReview creator feedback</h1>
<h1>hReview creator feedback</h1>



Revision as of 19:12, 3 January 2009

hReview creator feedback

This page is for collecting bugs, suggested fixes, and feature requests for the 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!


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) --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) --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