On 24.04.2011 17:43, Anurag Priyam wrote:
Hi,

Attached patch modifies the bundled rc.lua (awesomerc).

 From the commit message:
awesomerc: minimize/restore client on a left click in the tasklist entry

In the bundled rc.lua, mouse button 1 on a tasklist entry was configured to
bring the corresponding client to focus. Modify it to minimize the client if
already focussed.

Such (default) behavior seems more natural.

I guess I'm fine with this behavior, but I just managed to confuse myself (not your fault, but the existing code's fault).

Are you fine with changing the code like this?

 if c == client.focus then
     -- Clicking on a focused client minimizes it
     c.minimized = true
 else
    if not c:isvisible() then
        awful.tag.viewonly(c:tags()[1])
    end
    -- This will also un-minimize the client, if needed
    client.focus = c
    c:raise()
 end

I added two comments and changed "c.minimized = not c.minimized" into "c.minimized = true". If you are OK with this, I'd like to sneak this changes into your patch and let it look like you came up with this. :-)

Cheers,
Uli

--
"Do you know that books smell like nutmeg or some spice from a foreign land?"
                                                  -- Faber in Fahrenheit 451

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

Reply via email to