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

Re: [msysGit] Re: [PATCH] git-gui: offer a list of recent repositories on startup

From
Steffen Prohaska <prohaska@zib.de>
Date
Oct 10, 2007, 08:13 UTC
Message-ID
<B1EFFED4-1DF7-4A0A-93E2-F21812E7AF6B@zib.de>
In-Reply-To
<20071010073053.GQ2137@spearce.org>
Shawn,
On Oct 10, 2007, at 9:30 AM, Shawn O. Pearce wrote:
Show 21 quoted lines
>
> Steffen Prohaska <prohaska@zib.de> wrote:
>> Shawn,
>> I attached two patches. They should eventually be both squashed into
>> one.
>
> Thanks.  I'm applying the patch below over both of yours, and
> then squashing them all together and keeping you as the author.
>
> I did not like how the listbox looked (or worked!) on Mac OS X.
> Clicking on the item was at first a very non-obvious action, and
> then when I did do it I was very shocked to see the repository open
> immediately.  My brain knew the action was wired to the selection
> event (I obviously read the two patches before applying them),
> but it didn't expect that behavior.  Clearly the wrong UI model.
>
> The patch below uses a text widget instead and renders each of
> the repositories as a blue underlined link.  Users will generally
> assume that clicking on such links will take them immediately to
> that repository (much as it would take them to another webpage if
> this was a web browser)

This is similar to what I had in mind, but was lacking tk knowledge to implement it.

Show 19 quoted lines
> The "Open Recent Repositories" is not even shown in the UI if there
> aren't any to offer to the user.  Showing them this area just looks
> silly if they haven't opened or cloned anything yet.
>
> We now add freshly created or cloned repositories to the list of
> recently opened repositories.  These are just as fair game for
> being recently accessed as any others.  Perhaps even more so as
> users may wonder where they cloned that repository to the last time
> they started git-gui.
>
> We always store the absolute path of a repository into the config.
> This avoids the mess of opening a repository twice using say a
> relative path entered into the text entry field and then opening
> it later via a file browser, only to find out you now have the same
> repository in the recent list twice.
>
> I fixed the --unset-all bug and also avoided the --unset-all/--add
> loop you were using.  We convert the value into a "safe" expression
> and then try to match on it.
That all sound very reasonable to me.
I haven't applied your patch but only read through it. Looks good to me.

Thanks, Steffen

Previous: Shawn O. PearceNext: Shawn O. Pearce
Message 9 of 10 in “git-gui: offer a list of recent repositories on startup”
  1. git-gui: offer a list of recent repositories on startupSteffen Prohaska, Oct 7, 2007
  2. Shawn O. PearceOct 7, 2007
  3. Steffen ProhaskaOct 8, 2007
  4. Johannes SchindelinOct 9, 2007
  5. Steffen ProhaskaOct 9, 2007
  6. Shawn O. PearceOct 10, 2007
  7. Johannes SchindelinOct 10, 2007
  8. Shawn O. PearceOct 10, 2007
  9. Steffen ProhaskaOct 10, 2007
  10. Shawn O. PearceOct 10, 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.