Re: [wmii] Re: latest changes

From: Alex Kilgore <adkilgore_AT_gmail.com>
Date: Wed, 15 Oct 2008 20:48:00 -0400

        Thanks, that diff worked just fine, the changes look really good so
far, I really like the idea of having support for Xinerama, ( i havent
been able to test that yet because i have no second monitor here ) but I
am looking forward to that as soon as i can get my second monitor out!
        After applying the diff that was provided in the previous message. the
only thing I see that is a little odd is that moving a window left of
the initial column crashes wmii, also so does toggling full screen on
and off several times in default mode (though I think that might have
happened before)
        Looks great!
                Alex

hiro wrote:
> On Wed, Oct 15, 2008 at 11:14 PM, hiro <23hiro_AT_googlemail.com> wrote:
>> Kris has done quite some changes;)
>> I'm testing now, I guess you'll have to apply this diff for making it work.
>>
>> diff -r 361ab709dd0e cmd/menu/menu.c
>> --- a/cmd/menu/menu.c Wed Oct 15 16:08:56 2008 -0400
>> +++ b/cmd/menu/menu.c Wed Oct 15 23:08:53 2008 +0200
>> @@ -246,8 +246,8 @@
>> if(IsFunctionKey(ksym)
>> || IsMiscFunctionKey(ksym)
>> || IsKeypadKey(ksym)
>> - || IsPrivateKeypadKey(ksym))
>> - || IsPFKey(ksym)
>> + || IsPrivateKeypadKey(ksym)
>> + || IsPFKey(ksym))
>> return;
>>
>> if(e->state & ControlMask) {
>>
>
> Oh, I just saw, that isPFKey(kesym) appears multiple times now. it's
> rather just
> - || IsPrivateKeypadKey(ksym))
> - || IsPFKey(ksym)
> + || IsPrivateKeypadKey(ksym)
>
>
Received on Thu Oct 16 2008 - 00:48:00 UTC

This archive was generated by hypermail 2.2.0 : Thu Oct 16 2008 - 01:00:12 UTC