On Fri, 10 Sep 2010 at 20:51:04 +0400, Alexey I. Froloff wrote:
> Completely disable clip caption.

>  src/WindowMaker.h |    2 ++
>  src/defaults.c    |    2 ++
>  src/dock.c        |    3 ++-
>  3 files changed, 6 insertions(+), 1 deletions(-)

Hm, there is no code added in WPrefs to make the choice
available(?)

> --- a/src/defaults.c
> +++ b/src/defaults.c
> @@ -472,6 +472,8 @@ WDefaultEntry optionList[] = {
>           NULL, getFont, setIconTitleFont, NULL, NULL},
>       {"ClipTitleFont", DEF_CLIP_TITLE_FONT, NULL,
>           NULL, getFont, setClipTitleFont, NULL, NULL},
> +     {"ShowClipTitle", "YES", NULL,
> +             &wPreferences.show_clip_title, getBool, NULL, NULL, NULL},

Shouldn't it be "NO"?

> --- a/src/dock.c
> +++ b/src/dock.c
> @@ -1183,7 +1183,8 @@ void wClipIconPaint(WAppIcon * aicon)
>  
>       tx = CLIP_BUTTON_SIZE * ICON_SIZE / 64;
>  
> -     WMDrawString(scr->wmscreen, win, color, scr->clip_title_font, tx, ty, 
> ws_name, length);
> +     if(wPreferences.show_clip_title)
> +             WMDrawString(scr->wmscreen, win, color, scr->clip_title_font, 
> tx, ty, ws_name, length);

Unless I am missing something, there is no place where you can 
change show_clip_title, so what's the point of having the if()?
Why don't you simply remove the WMDrawString() line?


-- 
To unsubscribe, send mail to [email protected].

Reply via email to