comparison src/w32term.c @ 76016:dea718e903a0

(w32_set_scroll_bar_thumb): Don't update thumb size while dragging to avoid "random scrolling" effects. Disable unused code.

76015:ab0af4540bf8

  double range = VERTICAL_SCROLL_BAR_TOP_RANGE (f, XINT (bar->height));
  int sb_page, sb_pos;
  BOOL draggingp = !NILP (bar->dragging) ? TRUE : FALSE;
  SCROLLINFO si;

  if (whole)
    {
      /* Position scroll bar at rock bottom if the bottom of the
         buffer is visible. This avoids shinking the thumb away
         to nothing if it is held at the bottom of the buffer.  */
      if (position + portion >= whole)
        {
          sb_page = range * (whole - position) / whole
            + VERTICAL_SCROLL_BAR_MIN_HANDLE;
          sb_pos = range;
        }

      sb_page = portion * range / whole + VERTICAL_SCROLL_BAR_MIN_HANDLE;
      sb_pos = position * range / whole;
    }
  else

76016:dea718e903a0

  double range = VERTICAL_SCROLL_BAR_TOP_RANGE (f, XINT (bar->height));
  int sb_page, sb_pos;
  BOOL draggingp = !NILP (bar->dragging) ? TRUE : FALSE;
  SCROLLINFO si;

  /* Dragging the w32 scroll-bar handle in a file like INSTALL.CVS
     gives very odd scrolling behaviour with the code below which
     "only updates page size if currently dragging".  The cause
     seems to be that the W32 api may "silently" adjust the thumb
     position when the thumb page is change.

     So simply don't make any updates to the scroll-bar geometry
     while dragging.  KFS 2007-02-19.  */

  if (draggingp)
    return;

  if (whole)
    {
#if 0
      /* This code is not used (the settings are overwritten
	 immediately by the lines below it).
	 Should it be used?  KFS 2007-02-19.  */

      /* Position scroll bar at rock bottom if the bottom of the
         buffer is visible. This avoids shinking the thumb away
         to nothing if it is held at the bottom of the buffer.  */
      if (position + portion >= whole)
        {
          sb_page = range * (whole - position) / whole
            + VERTICAL_SCROLL_BAR_MIN_HANDLE;
          sb_pos = range;
        }
#endif

      sb_page = portion * range / whole + VERTICAL_SCROLL_BAR_MIN_HANDLE;
      sb_pos = position * range / whole;
    }
  else