Re: [pgAdmin4] [Patch]: Grant Wizard - Mailing list pgadmin-hackers

From Surinder Kumar
Subject Re: [pgAdmin4] [Patch]: Grant Wizard
Date
Msg-id CAM5-9D9PgDyCfg=B6MS=hPh0ZPBS+jtgcBLh6PCSyuhmrCv7pQ@mail.gmail.com
Whole thread Raw
In response to Fwd: [pgAdmin4] [Patch]: Grant Wizard  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
List pgadmin-hackers
Hi,

PFA updated patch

Change: removed 'ajs_content' css from wizard.css causing padding issue in alertify dialog.
Thanks murtuza for reporting.


On Fri, May 6, 2016 at 3:00 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi,

PFA patch with following issues fixed:
1. As SqlTab Control is renamed to SqlCtrl, throws error on grant wizard close. it is Fixed.
2. Moved grant wizard specific css from wizard.css to grant_wizard.css.

On Wed, Apr 13, 2016 at 8:41 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Fri, Apr 8, 2016 at 6:59 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

PFA patch with resolved review comments.

On Fri, Apr 8, 2016 at 12:52 AM, Dave Page <dpage@pgadmin.org> wrote:
Hi

Nearly there :-). Assuming no regressions, I believe we'll be ready to commit once the following issues are resolved:

- The select object grid should fill the available vertical space in the dialogue - at present there's a gap below it.
Done 

This does not seem to be fixed (see attached screenshot). In fact, the gap gets bigger proportionally as the window is resized.
Fixed 
 

- When selecting privileges, each time I click on a checkbox, the row closes. Similar grids elsewhere in the app close the row when the cell loses focus, *however*, that is also the incorrect behaviour - the row should only close when the row itself loses focus (and should open when it gets focus).
I checked the row closes either when gets clicked on row or outside row, it doesn't seems to close on click on checkbox.

This is still broken for me. I'm using Chrome on OS X. As soon as I click any checkbox, the row closes, and I have to click again to see the checkboxes again.
I checked with Chrome on OS X, windows and ubuntu, it is working fine. Can you please send me screenshot if possible?

One additional issue:

- If I select some objects and permissions (in my case, everything in a PEM database, and ALL), hit Next so I can see the SQL, then hit back *twice*, I see the second attached screenshot.
I followed the same steps and when hit back twice on previous button of SQL page, it takes me to first page which is right.
It would be easy for me to find out the exact cause if you send me output of browser console panel.

I'm going to commit the code with these issues as they are largely cosmetic. Please submit a patch to fix them (I'll add a new card to our internal Kanban chart).

Thanks.

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

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



Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fix menu width.
Next
From: Surinder Kumar
Date:
Subject: [pgAdmin4][Patch]: Load/Save file in query tool