Skip to content

Instantly share code, notes, and snippets.

@legumbre
Created August 14, 2013 00:32
Show Gist options
  • Save legumbre/6227025 to your computer and use it in GitHub Desktop.
Save legumbre/6227025 to your computer and use it in GitHub Desktop.

Weird scroll-top-left behavior

/*
 * Conkeror 1.0pre (XULRunner 23.0, Darwin)
 * 
 */       
function scroll_horiz_complete (buffer, n) {
    var w = buffer.focused_frame;
    dumpln("scroll_horiz_complete | before w.ScrollTo()| scrollX: " + w.scrollX);
    dumpln("scroll_horiz_complete | before w.ScrollTo()| scrollY: " + w.scrollY);
    w.scrollTo (n > 0 ? w.scrollMaxX : 0, w.scrollY);
    cdumpln("scroll_horiz_complete | after  w.ScrollTo() | scrollX: " + w.scrollX);
    dumpln("scroll_horiz_complete | after  w.ScrollTo() | scrollY: " + w.scrollY);
}

interactive("scroll-top-left",
    "Scroll the current frame all the way to the top left",
     function (I) { 
         var w= I.buffer.focused_frame;
         dumpln("scroll-top-left | before do_command(cmd_scrollTop)|  scrollX " + w.scrollX);
         dumpln("scroll-top-left | before do_command(cmd_scrollTop)|  scrollY " + w.scrollY);
         I.buffer.do_command("cmd_scrollTop");
         dumpln("scroll-top-left | after  do_command(cmd_scrollTop)|  scrollX " + w.scrollX);
         dumpln("scroll-top-left | after  do_command(cmd_scrollTop)|  scrollY " + w.scrollY);
         scroll_horiz_complete(I.buffer, -1); });

Test A Output

scroll-top-left | before do_command(cmd_scrollTop)|  scrollX 0
scroll-top-left | before do_command(cmd_scrollTop)|  scrollY 215
scroll-top-left | after  do_command(cmd_scrollTop)|  scrollX 0
scroll-top-left | after  do_command(cmd_scrollTop)|  scrollY 215  <-- I would expect scrollY to be 0
scroll_horiz_complete | before w.ScrollTo()| scrollX: 0
scroll_horiz_complete | before w.ScrollTo()| scrollY: 215         <-- w.scrollTo apparently scrolls to (0, 215) which is where we started.
scroll_horiz_complete | after  w.ScrollTo() | scrollX: 0
scroll_horiz_complete | after  w.ScrollTo() | scrollY: 215
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment