git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jakub Narebski <jnareb@gmail.com>
To: git@vger.kernel.org
Subject: [WORKAROUND] gitk lower pane scrollbar extends past gitk window
Date: Thu, 22 Jun 2006 09:48:52 +0200	[thread overview]
Message-ID: <e7di0n$7hh$1@sea.gmane.org> (raw)
In-Reply-To: 81b0412b0606220010p6c2e62ffv3914abafd95e4bb2@mail.gmail.com

Alex Riesen wrote:

> On 6/22/06, Edgar Toernig <froese@gmx.de> wrote:
>> Jakub Narebski wrote:
>>>
>>> In gitk from the current 'next' branch, post git version 1.4.0
>>> (blob ba4644f) scrollbar for lower pane, i.e. for commitdiff and files
>>> (Comments) views extends past the bottom of the gitk window. Therefore
>>> I cannnot see lower part of commit diff if it is larger than 
>>> window height. 
>>
>> Yes, and the search field at the bottom is invisible too.  Removing
>> line 431:
>>
>>         .ctop conf -width $geometry(width) -height $geometry(height)
>>
>> seems to fix it and the window still gets the right size.
>>
> 
> No, it does not. It gets the default geometry, not the one it had last time.

Now I see. It is the case when I used gitk from a computer with a desktop
with larger screen height, then on computer with screen height smaller than 
gitk height was I had mentioned problem.

Workaround for now is to delete old geometry info from ~/.gitk file 
(or just delete whole file).

I don't know Tcl/Tk to make proper fix, i.e. check if restored size is not
larger than available screen size, and use min(saved size, screen size).
Another improvement would be to save the fact that gitk window is maximized
as a symbol, e.g. as equivalent of 

  set geometry(height) screen(height)

in ~/.gitk

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git

  reply	other threads:[~2006-06-22  7:49 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-21 12:42 gitk lower pane (commit and files view) scrollbar extends past gitk window Jakub Narebski
2006-06-22  0:35 ` Edgar Toernig
2006-06-22  6:52   ` [PATCH] gitweb: whitespace cleanup around '=' Jakub Narebski
2006-06-22  7:01     ` Jakub Narebski
2006-06-22  7:09     ` Junio C Hamano
2006-06-22  7:32       ` Jakub Narebski
2006-06-22  9:33         ` Junio C Hamano
2006-06-22  9:37           ` Jakub Narebski
2006-06-22 10:20             ` Junio C Hamano
2006-06-22  9:52           ` Jakub Narebski
2006-06-22 10:19             ` Santi Béjar
2006-06-22  7:10   ` gitk lower pane (commit and files view) scrollbar extends past gitk window Alex Riesen
2006-06-22  7:48     ` Jakub Narebski [this message]
2006-06-22 13:19       ` [WORKAROUND] gitk lower pane " Jakub Narebski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='e7di0n$7hh$1@sea.gmane.org' \
    --to=jnareb@gmail.com \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).