3 ms·

Thanks a ton for the feedback Jolan, I've deployed some of your suggestions. We've marked the reason why we need the email address, if it let you see the repor

Thanks a ton for the feedback Jolan, I've deployed some of your suggestions. We've marked the reason why we need the email address, if it let you see the report without an email address, it's a bug :).. we'll fix it. I've merged the work on and full reports together. We're looking into the too many meta keywords issue, I'll fix that soon it's not a off by one issue, but I'm calculating words, instead of phrases. We're working on display issues :). I'm a former sysadmin who's now doing UI :) I need to get the images to show up in the copy better. We're running your CSS against Jigsaw and just outputting the result. One of the things that's on the way is "importance", so that the list of things to fix is displayed in order of importance. This would mean the css errors would be fairly low on the list.

jolan 16y ago
> We've marked the reason why we need the email address, if it let you see the report without an email address, it's a bug :).. we'll fix it. I really wouldn't do that but it's not my choice to make :) > I've merged the work on and full reports together. I saw that. Makes it a ton better. > We're running your CSS against Jigsaw and just outputting the result. I figured as much. Not a big deal. It might be nice to include the link in the report. One thing you might want to consider is building the report in real time rather than doing the status bar and then asking the user to click for the report. You can do something like the following w/jQuery: $.ajax({ url: '/api/v1/', data: { cmd: "report_getdata", id: reportid }, timeout: 5000, success: function(data) { // remove everything but the table header $("#reportinfo").find("tr:gt(0)").remove(); var obj = jQuery.parseJSON(data); jQuery.each(obj, function(i, ddata) { $("#reportinfo").last().append("<tr><td colspan=7>JSON report data row</td></tr>"); }); } });
- zippykid 16y ago
  You're reading my mind :). On each report item I'm going to have a "re-check" button so you can make a change and just analyze that one part. Shouldn't be too hard to make the report in real time.

jolan 16y ago

> We've marked the reason why we need the email address, if it let you see the report without an email address, it's a bug :).. we'll fix it. I really wouldn't do that but it's not my choice to make :) > I've merged the work on and full reports together. I saw that. Makes it a ton better. > We're running your CSS against Jigsaw and just outputting the result. I figured as much. Not a big deal. It might be nice to include the link in the report. One thing you might want to consider is building the report in real time rather than doing the status bar and then asking the user to click for the report. You can do something like the following w/jQuery: $.ajax({ url: '/api/v1/', data: { cmd: "report_getdata", id: reportid }, timeout: 5000, success: function(data) { // remove everything but the table header $("#reportinfo").find("tr:gt(0)").remove(); var obj = jQuery.parseJSON(data); jQuery.each(obj, function(i, ddata) { $("#reportinfo").last().append("<tr><td colspan=7>JSON report data row</td></tr>"); }); } });

zippykid 16y ago
You're reading my mind :). On each report item I'm going to have a "re-check" button so you can make a change and just analyze that one part. Shouldn't be too hard to make the report in real time.