Re: checklink: ideas for next version

On Tue, Jan 6, 2009 at 4:22 PM, olivier Thereaux <ot@w3.org> wrote:
>
> I have not yet had the time to go through testing and bugzilla, but the
> (small, and not as good as I'd have liked) style/usability changes are now
> in CVS.
>
> Ville (and all), can you review the changes and give your OK?
>
> Can be tested at:
> http://qa-dev.w3.org/wlc/checklink?uri=http%3A%2F%2Fqa-dev.w3.org

My apologies for being absent... I'm just getting caught up from being
away for the holidays.  :-|  So, by all means don't wait on me if
you'd like to get a release sent out.

Regarding what you have in cvs now: I think the style changes are
looking great. After trying it out, the first thing I wanted was
auto-scrolling for the progress section.

I don't believe I have access to the repository, but I'll gladly send patches.

This 3 line patch just auto scrolls the progress section:

### Eclipse Workspace Patch 1.0
#P LinkChecker
Index: bin/checklink
===================================================================
RCS file: /sources/public/perl/modules/W3C/LinkChecker/bin/checklink,v
retrieving revision 4.131
diff -u -r4.131 checklink
--- bin/checklink	6 Jan 2009 22:19:33 -0000	4.131
+++ bin/checklink	7 Jan 2009 15:13:04 -0000
@@ -333,6 +333,8 @@
 oNewChild=document.createElement("span");
 oNewChild.appendChild(txt);
 progressHead.replaceChild(oNewChild, oChild);
+var progress = document.getElementById("progress");
+progress.scrollTop = progress.scrollHeight+10;
 };</script>
 EOF

@@ -919,7 +921,7 @@
              &encode($Opts{_Self_URI}));

       printf ('<h3 class="status_progress"
id="status_progress%s">Status: <span></span></h3>', $result_anchor);
-      print("<pre class=\"progress\">\n");
+      print("<pre class=\"progress\" id=\"progress\">\n");

     }
   }


-- 
Brett Bieber
Office of University Communications
University of Nebraska-Lincoln

Received on Wednesday, 7 January 2009 15:22:37 UTC