Thanks, applied! On Thu, Jun 8, 2017 at 4:53 PM, Surinder Kumar <surinder.ku...@enterprisedb.com> wrote: > Hi > > I > have > replaced Silver, #222, and #aaa colors > with relevant > colors > defined in pgadmin.styles.css > Also, the header rows > left & > right > border > color is changed to #888 to match with bottom > border > (#888) > , now the header can easily be distinguished from the grid rows border > styling. > > Please review and let me know if any changes. > > > On Thu, Jun 8, 2017 at 7:51 PM, Dave Page <dp...@pgadmin.org> wrote: >> >> Exactly what my review was going to say :-) >> >> On Thu, Jun 8, 2017 at 3:07 PM, Sarah McAlear <smcal...@pivotal.io> wrote: >> > Hi Surinder! >> > >> > We reviewed the patch and the changes look great. Could you change the >> > grays >> > to match the style guide? >> > >> > Silver, #222, and #aaa aren't listed in pgadmin.style.css. >> > >> > Thanks so much! >> > Shirley & Sarah >> > >> > On Thu, Jun 8, 2017 at 9:34 AM, Surinder Kumar >> > <surinder.ku...@enterprisedb.com> wrote: >> >> >> >> Hi, >> >> >> >> Changes: >> >> >> >> 1) Display only border-top for data-grid header instead of border on >> >> whole >> >> data-grid. >> >> >> >> 2) Remove dotted border from cell right for consistency. >> >> >> >> 3) Add border-bottom for header-row so that on scroll to bottom header >> >> rows are differentiated from cells. >> >> >> >> Also, due to these changes, the vertical scrollbar which appears at >> >> data-grid right side seems to fixed. >> >> >> >> PFA patch and review. >> >> >> >> >> >> Thanks, >> >> Surinder >> >> >> >> >> >> -- >> >> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >> >> To make changes to your subscription: >> >> http://www.postgresql.org/mailpref/pgadmin-hackers >> >> >> > >> >> >> >> -- >> Dave Page >> Blog: http://pgsnake.blogspot.com >> Twitter: @pgsnake >> >> EnterpriseDB UK: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company > >
-- 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