{"thread":{"id":"47423","subject":"[PATCH] git-gui: Make push remote combobox full width","startedAt":"2017-12-11T01:13:51Z","lastAt":"2017-12-18T02:58:47Z","messageCount":3,"participants":["Peter Urbanec","Eric Sunshine"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"334588","messageId":"20171211004848.13048-1-git.user@urbanec.net","threadId":"47423","inReplyTo":null,"subject":"[PATCH] git-gui: Make push remote combobox full width","fromName":"Peter Urbanec","fromEmail":"git.user@urbanec.net","sentAt":"2017-12-11T00:48:48Z","receivedAt":"2017-12-11T01:13:51Z","isPatch":true,"sender":{"key":"git.user@urbanec.net","avatar":null},"body":"When pushing changes, the Remote combobox for the Destination Repository\ndoes not take all available space in the layout. With long remote names,\nthis causes the combobox to have truncated entries even though there is\nroom to display them.\n\nAnchor the remote_m combobox to both left and right edges of the grid\ncell and use the same padding as url_t for a consistent, dynamic layout.\n\n\nSigned-off-by: Peter Urbanec <git.user@urbanec.net>\n---\n lib/transport.tcl | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/lib/transport.tcl b/lib/transport.tcl\nindex a1a424a..1046dc5 100644\n--- a/lib/transport.tcl\n+++ b/lib/transport.tcl\n@@ -173,7 +173,7 @@ proc do_push_anywhere {} {\n \t\t} else {\n \t\t\teval tk_optionMenu $w.dest.remote_m push_remote $all_remotes\n \t\t}\n-\t\tgrid $w.dest.remote_r $w.dest.remote_m -sticky w\n+\t\tgrid $w.dest.remote_r $w.dest.remote_m -sticky we -padx {0 5}\n \t\tif {[lsearch -sorted -exact $all_remotes origin] != -1} {\n \t\t\tset push_remote origin\n \t\t} else {\n-- \n2.15.1\n\n"},{"id":"334949","messageId":"7aff934e-c26a-8259-17e0-436104f377cf@urbanec.net","threadId":"47423","inReplyTo":"20171211004848.13048-1-git.user@urbanec.net","subject":"Re: [PATCH] git-gui: Make push remote combobox full width","fromName":"Peter Urbanec","fromEmail":"git.user@urbanec.net","sentAt":"2017-12-17T13:49:00Z","receivedAt":"2017-12-17T14:33:40Z","isPatch":true,"sender":{"key":"git.user@urbanec.net","avatar":null},"body":"Just wondering if I submitted this patch correctly and to the right place.\n\nOn 11/12/17 11:48, Peter Urbanec wrote:\n> When pushing changes, the Remote combobox for the Destination Repository\n> does not take all available space in the layout. With long remote names,\n> this causes the combobox to have truncated entries even though there is\n> room to display them.\n> \n> Anchor the remote_m combobox to both left and right edges of the grid\n> cell and use the same padding as url_t for a consistent, dynamic layout.\n> \n> \n> Signed-off-by: Peter Urbanec <git.user@urbanec.net>\n> ---\n>   lib/transport.tcl | 2 +-\n>   1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/lib/transport.tcl b/lib/transport.tcl\n> index a1a424a..1046dc5 100644\n> --- a/lib/transport.tcl\n> +++ b/lib/transport.tcl\n> @@ -173,7 +173,7 @@ proc do_push_anywhere {} {\n>   \t\t} else {\n>   \t\t\teval tk_optionMenu $w.dest.remote_m push_remote $all_remotes\n>   \t\t}\n> -\t\tgrid $w.dest.remote_r $w.dest.remote_m -sticky w\n> +\t\tgrid $w.dest.remote_r $w.dest.remote_m -sticky we -padx {0 5}\n>   \t\tif {[lsearch -sorted -exact $all_remotes origin] != -1} {\n>   \t\t\tset push_remote origin\n>   \t\t} else {\n> \n\n"},{"id":"334957","messageId":"CAPig+cRSPyR9dwbA7rHhYGorKQFr2Kh64PQtgQK3nxoQXvUGhw@mail.gmail.com","threadId":"47423","inReplyTo":"7aff934e-c26a-8259-17e0-436104f377cf@urbanec.net","subject":"Re: [PATCH] git-gui: Make push remote combobox full width","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2017-12-18T02:58:41Z","receivedAt":"2017-12-18T02:58:47Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sun, Dec 17, 2017 at 8:49 AM, Peter Urbanec <git.user@urbanec.net> wrote:\n> Just wondering if I submitted this patch correctly and to the right place.\n\nYou sent it to the right place. Unfortunately, Pat hasn't been around\nmuch lately, so the git-gui project is apparently slumbering.\n\nJunio has accepted a few patches for git-gui recently[1,2,3]; whether\nhe's interested in doing the same for your patch is unknown.\n\n[1]: https://github.com/patthoyts/git-gui/pull/17\n[2]: https://github.com/patthoyts/git-gui/pull/16\n[3]: https://github.com/patthoyts/git-gui/pull/15\n"}]}