Re: [pgadmin-hackers] [patch] Query Output Header Width - Mailing list pgadmin-hackers

From Sarah McAlear
Subject Re: [pgadmin-hackers] [patch] Query Output Header Width
Date
Msg-id CAGRPzo9WUy4CAGCzhzNm-R2ak-rm1sBKFLBCmO85Vw8MgNdCmw@mail.gmail.com
Whole thread Raw
In response to Re: [pgadmin-hackers] [patch] Query Output Header Width  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgadmin-hackers] [patch] Query Output Header Width  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Oh, good catch! Here's a new patch with all the lines in place.

Thanks,
Joao & Sarah

On Thu, Apr 20, 2017 at 12:05 PM, Dave Page <dpage@pgadmin.org> wrote:


On Fri, Apr 14, 2017 at 11:01 PM, George Gelashvili <ggelashvili@pivotal.io> wrote:
Hello Hackers,

We found that having the column header extend all the way across the query output frame made it seem like there were more columns. This was confusing to some users.

We made a change to the column header styling so it ends with the rightmost column.

Happy Friday!
Matt & Shirley & George

Hi Matt & Shirley & George!

I agree with the basic premise of this patch, however it has the side-effect of removing the bottom border from the entire header row, which makes the transition from the header ro to the first data row look very odd (especially in the first row header, which essentially merges into the top-left "select all" cell). Screenshot attached.

Can you get the line back in place, but so it only shows up under the actual header cells?

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers


Attachment

pgadmin-hackers by date:

Previous
From: Sarah McAlear
Date:
Subject: Re: [pgadmin-hackers][patch] Move to Alembic migration system
Next
From: Dave Page
Date:
Subject: [pgadmin-hackers] pgAdmin 4 commit: Only display the header row in SlickGrid wherecolumn