hreview-creator-issues: Difference between revisions
Jump to navigation
Jump to search
(added lines of code) |
|||
| Line 14: | Line 14: | ||
<pre>Error: pad is not defined | <pre>Error: pad is not defined | ||
Source File: http://microformats.org/code/hreview/creator | Source File: http://microformats.org/code/hreview/creator | ||
Line: 230</pre> | Line: 230 | ||
$('#review-date-day').val(pad(d.getDate()));</pre> | |||
* Also JavaScript error when page is submitted --[[User:TravisHardiman|TravisHardiman]] 08:16, 4 Feb 2008 (PST) | * Also JavaScript error when page is submitted --[[User:TravisHardiman|TravisHardiman]] 08:16, 4 Feb 2008 (PST) | ||
<pre>Error: format_dt is not defined | <pre>Error: format_dt is not defined | ||
Source File: http://microformats.org/code/hreview/creator | Source File: http://microformats.org/code/hreview/creator | ||
Line: 92</pre> | Line: 92 | ||
var dt = format_dt(review_date_year, review_date_month, review_date_day, review_date_hour, review_date_minute, timezone);</pre> | |||
== Suggested Tweaks and Fixes == | == Suggested Tweaks and Fixes == | ||
Revision as of 16:22, 4 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 didn't noticed value of url field in generated code. reported by Dmitry Baranovskiy at 2006-07-20 (link to mail archives?)
- 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
- ...