Re: [tig PATCH v2 0/2] popup menu

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Feb 25, 2009 at 11:32 PM, Jonas Fonseca <jonas.fonseca@xxxxxxxxx> wrote:
> On Sun, Feb 22, 2009 at 15:09, Giuseppe Bilotta
> <giuseppe.bilotta@xxxxxxxxx> wrote:
>> Second version, in two patches. The first one is just the same as
>> previous one, minus a couple of leftovers.
>>
>> The second one reimplements the menu drawing routine so that it can be
>> called when necessary to redraw the menu entirely, e.g. after a resize
>> or when background loading finishes.
>
> I didn't read them through very carefully yet, so will "queue" them
> for now until I get fully back to tig. There are still some quirks
> that might needs to be addressed related with resizing, but this could
> also be fixed later. Anyway, I would like to think a bit more whether
> or not I like the cute menus, and also whether options might be served
> better by introducing an option view.

I like the minimalistic interface of tig, but when the user has to
select among a fixed number of choices it's much better to be able to
see all of them. The idea of a menu view which is not a popup is
probably good too, but it might have issues with the switching (what
if q is a hotkey? how would you close the view?)

> Regarding the "tig branch", would you mind if I postpone applying it
> until the branch view develops into something that is actually usable?

Please, be my guest 8-) I'm not even convinced it's useful anymore 8-)
("tig branch", not the branch view).

-- 
Giuseppe "Oblomov" Bilotta
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux