hreview-creator-issues: Difference between revisions

From Microformats Wiki
Jump to navigation Jump to search
(added lines of code)
(collected bug report from Twitter)
Line 21: Line 21:
Line: 92
Line: 92
       var dt = format_dt(review_date_year, review_date_month, review_date_day, review_date_hour, review_date_minute, timezone);</pre>
       var dt = format_dt(review_date_year, review_date_month, review_date_day, review_date_hour, review_date_minute, timezone);</pre>
* http://twitter.com/fedmor/statuses/751276422 - hReview creator fails in Firefox


== Suggested Tweaks and Fixes ==
== Suggested Tweaks and Fixes ==

Revision as of 22:56, 27 February 2008

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