On Thu, Jan 28, 2016 at 6:01 AM, Michael Paquier <michael.paqu...@gmail.com> wrote:
> On Thu, Jan 28, 2016 at 1:54 PM, David G. Johnston > <david.g.johns...@gmail.com> wrote: > > On Wed, Jan 27, 2016 at 9:13 PM, Michael Paquier < > michael.paqu...@gmail.com> > > wrote: > >> > >> On Thu, Jan 28, 2016 at 9:34 AM, David G. Johnston > >> <david.g.johns...@gmail.com> wrote: > >> > So how about: > >> > > >> > + snprintf(title, strlen(myopt.title) + 50, > >> > + _("Watch every %lds\t%s\t%s"), > >> > + sleep, head_title, asctime(localtime(&timer))); > >> > >> I would just keep the timestamp and the title separated so what do you > >> think about that instead? > >> Watch every Xs $timestamp > >> $head_title > > > > > > That works. I like having the title immediately above the table. > > > > The other option that came to mind would be to place the time information > > after the table display while leaving the title before it. On an output > > that requires more vertical space than is available in the terminal one > > would no longer have to scroll up to confirm last execution time. If > doing > > this I'd probably get rid of any logic that attempts to center the time > > information on the table and simply leave it left-aligned. > > OK, attached is an updated patch. How does that look? > > Could you run the following and post the result, please? I have yet to setup a compiled-from-source installation... \pset title 'This is a medium length title' SELECT repeat('a', 100) \watch 5 It seems fine though I'm curious how it plays with various combinations of narrow and wide outputs on narrow and wide terminals - specifically the centering aspect that occurs when the table is wider than the Watch/Time content. The terminal width should just cause wrapping relative to a wide-enough terminal so really it just the centering aspect when the table is wider than the header. Thanks! David J.