git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Improved git-gui blame viewer

From
MMMatthijs Melchior <mmelchior@xs4all.nl>
Date
Jun 5, 2007, 21:47 UTC
Message-ID
<4665D9E9.8050409@xs4all.nl>
In-Reply-To
<20070605042855.GA9513@spearce.org>
Shawn O. Pearce wrote:
Show 39 quoted lines
> Matthijs Melchior <mmelchior@xs4all.nl> wrote:
>   
>> Shawn O. Pearce wrote:
>>     
>>> I'm not sure I understand what you are looking for here.  Right now
>>> git-gui should be inverting the foreground/background colors on
>>> the file that is "selected" (shown in the lower diff view pane).
>>> So the background should be black, and the foreground white.
>>> Is this not happening?  Or are you looking for something else?
>>>
>>>       
>> No, I am not looking for something else...., the inverting you describe
>> does not happen on my machine....
>>     
>
> I'm wrong.  Its not inverting.  Its bold if its selected, and normal
> if its not selected.  Perhaps your font is already a bold weight
> so you aren't seeing a difference between the selected item and
> the non-selected items.
>  
>   
>> I am now running Debian git-core 1.5.2.1-1 with 'make install' done
>> in the origin/pu branch of git-gui.
>> 'About git-gui' now says:
>> 	git-gui version 0.7.2.58-gf9e9
>> 	git version 1.5.2.1
>> 	Tcl/Tk version 8.4.12
>>
>> If you explain where this inverting is taking place, I can do some
>> experiments to find out more [use gray background i.s.o. inverting...]
>> Maybe it has something to do with Desktop themes, I use the standard
>> Gnome theme.
>>     
>
> Around line 1803 of git-gui.sh we setup the in_diff tag for the
> $ui_index and $ui_workdir Tk widgets.  That tag is applied to the
> file that is in the diff viewer.  Perhaps adding a background to
> the tag would get you an improved interface?
>   
Yes, this is the problem.

I will send you a patch to change the background of the selected file to lightgray.

I have included in that patch some softer colors for the headers of the three involved widgets. I hope you like these.

Thanks.
-- 
Regards,
----------------------------------------------------------------  -o)
Matthijs Melchior                                       Maarssen  /\\
mmelchior@xs4all.nl                                  Netherlands _\_v
---------------------------------------------------------------- ----
Previous: Shawn O. PearceNext: Alex Riesen
Message 9 of 12 in “Improved git-gui blame viewer”
  1. Shawn O. PearceJun 2, 2007
  2. Matthijs MelchiorJun 2, 2007
  3. Shawn O. PearceJun 4, 2007
  4. Martin WaitzJun 4, 2007
  5. Shawn O. PearceJun 4, 2007
  6. Martin WaitzJun 4, 2007
  7. Matthijs MelchiorJun 4, 2007
  8. Shawn O. PearceJun 5, 2007
  9. Matthijs MelchiorJun 5, 2007
  10. Alex RiesenJun 4, 2007
  11. Shawn O. PearceJun 5, 2007
  12. Alex RiesenJun 5, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.