Re: [pgadmin-hackers][patch] delete icon width in security labels form - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgadmin-hackers][patch] delete icon width in security labels form
Date
Msg-id CA+OCxozF+q=rexMbV8ME_D24c235tAK3h_vkg0pXOh-Ah42NzQ@mail.gmail.com
Whole thread Raw
In response to [pgadmin-hackers][patch] delete icon width in security labels form  (Joao De Almeida Pereira <jdealmeidapereira@pivotal.io>)
List pgadmin-hackers
Hi

On Thu, Jul 6, 2017 at 4:13 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Hackers,

We found a discrepancy with the width of inserted columns in some backform tables. It is at least present in Create Table -> Security [tab] -> clicking [+] for new rows under each of Privileges and Security Labels. In this case, the cell containing the delete icon is spaced differently.

After briefly investigating, it looks like width_percent_xx classes on too many columns is to blame. We've attached a patch that resolves the immediate issue by removing width_percent_50 from the last column of two schemas. It feels like columns are overconstrained since by themselves width percentages sum to 100%, but there is behavior that inserts additional columns beyond those specified in the schema. For instance, the unshifts in Backform.UniqueColCollectionControl.showGridControl in backform.pgadmin.js

Let us know what you think of this removal of last-column constraint as an approach.

I think it's fine for a quick fix. Longer term, it may be worth coming up with some more intelligent sizing, such as fixed space for the control icons, then some sort of fixed unit for the rest, that can be used so that it recalculates actual ratios on the fly.

Thanks, patch applied!

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

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

pgadmin-hackers by date:

Previous
From: pgAdmin 4 Jenkins
Date:
Subject: Jenkins build is back to normal : pgadmin4-master-python33 #222
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fix handling of CREATE TABLE OF . Fixes #2534