{"thread":{"id":"4006","subject":"Fwd: More qgit defects 2/2","startedAt":"2006-04-30T08:19:58Z","lastAt":"2006-04-30T08:19:58Z","messageCount":1,"participants":["Marco Costalba"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"19279","messageId":"e5bfff550604300119s7afe7b0p1e808b85cab43f81@mail.gmail.com","threadId":"4006","inReplyTo":null,"subject":"Fwd: More qgit defects 2/2","fromName":"Marco Costalba","fromEmail":"mcostalba@gmail.com","sentAt":"2006-04-30T08:19:58Z","receivedAt":"2006-04-30T08:19:58Z","isPatch":false,"sender":{"key":"mcostalba@gmail.com","avatar":null},"body":"I would like to forward this thread to the list in case someone find\nit interesting and/or suggests improvements.\n\n---------- Forwarded message ----------\nFrom: Marco Costalba <mcostalba@gmail.com>\nDate: Apr 30, 2006 10:11 AM\nSubject: Re: More qgit defects\nTo: Pavel Roskin <proski@gnu.org>\nCc: ydirson@altern.org\n\n\nHi Pavel,\n>\n> I think the list items in the file window need a pop-up menu.  Without\n> it, the users would not be able to discover what e.g. the double click\n> would do.\n>\n\nyes, I agree.\n\n> More generally, I feel that something is wrong with the user interface.\n> I think gitk is doing it better with a single pane that has the comments\n> and the diff.  Splitting the window horizontally in three panes seems to\n> be too much, especially when one of the panes is used for diffs.\n>\n\nPlease Pavel pull from latest qgit repo. I just pushed a patch that,\nat least for my\nbrowsing style, fixes this and IMHO is better then gitk.\n\nPatch description is as follow:\n\"Usability enhancement: use 'P' key to quick toggle patch viewer\n\n   One of the limitation of patch browsing in qgit is the use of two\n   frames, one for revision logs and the other for patch viewer.\n\n   If patch viewer is docked the space is normally limited by revision\n   log frame. If patch viewer is undocked it is always at top level and\n   hides main view and revision logs.\n\n   So this patch adds a new key binding 'P' that hides/unhides diff\n   viewer window.\n\n   The intended use is simple: open patch viewer and undock it so to cover\n   a good chunk of screen space, then press 'P' key and diff viewer will\n   disappear.\n\n   Now you can browse the revisions as usual and toggle diff viewer with 'P'\n   key so to easily view both logs and patches without any space constrains.\n\"\n\nAfter a little bit of using I have found this way very natural, quick\nand, especially important for me, do not compromise patch view size.\n\nWhat I do is to size the undocked patch viewer on my left screen, and\ntoggle the P key, but other adjustment are possible, see:\n\nhttp://digilander.libero.it/mcostalba/qgit_P_1.png\nhttp://digilander.libero.it/mcostalba/qgit_P_2.png\n\nTip: let your mouse be always over main list view so that patch view\ndoes not steal the focus and you can browse only with keyboard arrows\nas well with new key bindings (see F1).\n\nLet me know what you think.\n\n               Marco\n"}]}