Re: [pgAdmin4][Patch]: RM-2859: Can't create new user - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgAdmin4][Patch]: RM-2859: Can't create new user
Date
Msg-id CAFOhELcsRShwkRFXBfUrprzmS=WqsytoHyD91jRnbBff_Pb2Gw@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: RM-2859: Can't create new user  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch]: RM-2859: Can't create new user
List pgadmin-hackers


On Mon, Nov 27, 2017 at 4:13 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Nov 27, 2017 at 10:39 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Nov 27, 2017 at 2:59 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Nov 27, 2017 at 9:19 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Nov 27, 2017 at 2:20 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Nov 27, 2017 at 5:25 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Dave,

On Fri, Nov 24, 2017 at 3:21 PM, Dave Page <dpage@pgadmin.org> wrote:


On Thu, Nov 23, 2017 at 10:43 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Thu, Nov 23, 2017 at 2:58 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Nov 23, 2017 at 5:03 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #2859:  Can't create new user.

The "User Management" dialogue footer was overlapping the back-grid table which has been fixed.

If my screen is too small, it now looks like the attached screenshot, which is really quite ugly.

If we don't leave the bottom blank space then in case of error the error-message will shown on the grid itself and user can't perform any task.
Please refer the attached screen-shot for the same.

Right, but we also can't have that space left blank like that. Can't we extend the scroll range of the grid? In other words, always include space for an extra row or so, so it can scroll above the error message, when, and only when a message is shown?

 
Please find the attached screen-shot, If we always include an extra row. 
Suggestion please.

I think that's much better, though still not ideal. What if we made the error messages closable like other notifications? 

The error-messages in pgAdmin 4 are not closable, so it will not go with the flow.

I meant to do it globally. 

Should I create the separate case for this? 

No, I don't think there's any need for that. 

Does it seem like it would solve the problem appropriately? 


It would lead us to more complexity
1. How can we keep track of the closed error messages for multiple fields?
2. We have validated backbone model on focus out/change, so we need to change the basic error model.


I have attached the patch for the RM #2859.

Thanks,
Khushboo

--
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: Revert inadvertant change to doc page
Next
From: pgAdmin 4 Jenkins
Date:
Subject: Build failed in Jenkins: pgadmin4-master-python27 #396