Re: Patch: Added select2cell editor for backgrid [pgAdmin4] - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Patch: Added select2cell editor for backgrid [pgAdmin4]
Date
Msg-id CA+OCxoxuP_KwJpAnXao7xNcJ1MKJ834jVMt_F1DVQ-iSND==pQ@mail.gmail.com
Whole thread Raw
In response to Re: Patch: Added select2cell editor for backgrid [pgAdmin4]  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
List pgadmin-hackers
I've reverted the change to overrides.css that this patch made as it was that that was causing the padding issue on backgrid that we noted this morning when looking at the debugger.

+}
+
+.select2-cell .select2-container > .backgrid th, .backgrid td
+{
+    padding: 0px;

Please fix in a less blanket way.

On Mon, Apr 4, 2016 at 11:07 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Thanks - committed with small changes.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


http://www.linkedin.com/in/asheshvashi


On Mon, Apr 4, 2016 at 1:09 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA updated patch.

Regards,
Murtuza




--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Fri, Apr 1, 2016 at 3:08 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Thu, Mar 31, 2016 at 12:27 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:

Hi,

We have added Select2Cell Editor for backgrid, eariler as it was not present & causing issue in rendering in subnode control.
Looks good to me, but - have some feedback.
1. Can you please mention - why do we need 'openOnEnter: false' in select2 options?
2. There is a difference between UI, when rendered from the editor, and from the Select2Cell. (I can see padding, when the Cell is rendered, but - not when editor is rendered.)

Can you please fix this two?

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


http://www.linkedin.com/in/asheshvashi



--
Regards,
Murtuza Zabuawala
EnterpriseDB: 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







--
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: Dave Page
Date:
Subject: pgAdmin 4 commit: Revert style change introduced in commit f7e4e8bea010
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Minor adjustment to the menu font sizing.