--- rxvt-unicode/src/scrollbar-xterm.C 2004/08/15 22:09:24 1.15 +++ rxvt-unicode/src/scrollbar-xterm.C 2006/02/20 22:42:01 1.22 @@ -1,10 +1,10 @@ -/*--------------------------------*-C-*---------------------------------* +/*----------------------------------------------------------------------* * File: scrollbar-xterm.C *----------------------------------------------------------------------* * * Copyright (c) 1997,1998 mj olesen * Copyright (c) 1999-2001 Geoff Wing - * Copyright (c) 2004 Marc Lehmann + * Copyright (c) 2004-2006 Marc Lehmann * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -32,49 +32,54 @@ const unsigned char x_stp_bits[] = { 0xaa, 0x55 }; int -rxvt_term::scrollbar_show_xterm (int update __attribute__ ((unused)), int last_top, int last_bot, int scrollbar_len) +rxvt_term::scrollbar_show_xterm (int update, int last_top, int last_bot, int scrollbar_len) { - int xsb = 0; - int sbwidth = scrollBar.width - 1; + int xsb = 0; + int sbwidth = scrollBar.width - 1; if ((scrollBar.init & R_SB_XTERM) == 0) { XGCValues gcvalue; scrollBar.init |= R_SB_XTERM; - gcvalue.stipple = XCreateBitmapFromData (display->display, scrollBar.win, + gcvalue.stipple = XCreateBitmapFromData (dpy, scrollBar.win, (char *)x_stp_bits, x_stp_width, x_stp_height); if (!gcvalue.stipple) rxvt_fatal ("can't create bitmap\n"); gcvalue.fill_style = FillOpaqueStippled; - gcvalue.foreground = pix_colors[Color_fg]; - gcvalue.background = pix_colors[Color_bg]; + gcvalue.foreground = pix_colors_focused[Color_fg]; + gcvalue.background = pix_colors_focused[Color_bg]; - xscrollbarGC = XCreateGC (display->display, scrollBar.win, + xscrollbarGC = XCreateGC (dpy, scrollBar.win, GCForeground | GCBackground | GCFillStyle | GCStipple, &gcvalue); - gcvalue.foreground = pix_colors[Color_border]; - ShadowGC = XCreateGC (display->display, scrollBar.win, GCForeground, &gcvalue); + gcvalue.foreground = pix_colors_focused[Color_border]; + ShadowGC = XCreateGC (dpy, scrollBar.win, GCForeground, &gcvalue); } - /* instead of XClearWindow (display->display, scrollBar.win); */ - xsb = (options & Opt_scrollBar_right) ? 1 : 0; - if (last_top < scrollBar.top) - XClearArea (display->display, scrollBar.win, - sb_shadow + xsb, last_top, - sbwidth, (scrollBar.top - last_top), False); - - if (scrollBar.bot < last_bot) - XClearArea (display->display, scrollBar.win, - sb_shadow + xsb, scrollBar.bot, - sbwidth, (last_bot - scrollBar.bot), False); + + if (update) + { + xsb = OPTION (Opt_scrollBar_right) ? 1 : 0; + if (last_top < scrollBar.top) + XClearArea (dpy, scrollBar.win, + sb_shadow + xsb, last_top, + sbwidth, (scrollBar.top - last_top), False); + + if (scrollBar.bot < last_bot) + XClearArea (dpy, scrollBar.win, + sb_shadow + xsb, scrollBar.bot, + sbwidth, (last_bot - scrollBar.bot), False); + } + else + XClearWindow (dpy, scrollBar.win); /* scrollbar slider */ - XFillRectangle (display->display, scrollBar.win, xscrollbarGC, + XFillRectangle (dpy, scrollBar.win, xscrollbarGC, xsb + 1, scrollBar.top, sbwidth - 2, scrollbar_len); - XDrawLine (display->display, scrollBar.win, ShadowGC, + XDrawLine (dpy, scrollBar.win, ShadowGC, xsb ? 0 : sbwidth, scrollBar.beg, xsb ? 0 : sbwidth, scrollBar.end); return 1;