Re: User management functionality patch [pgadmin4]

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: User management functionality patch [pgadmin4]
Date: 2016-06-03 11:09:34
Message-ID: CA+OCxow4KOG9mGP1GpO3OzE=Xt=QdKZUkPdiR6MdE_Y2jdwApw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Fri, Jun 3, 2016 at 11:37 AM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com
> wrote:

> Hi Harshal,
>
> Dave asked to put the User Management menu under the 'Change Password'
> (right top side).
>

Correct - that way it won't be displayed in desktop mode. Other comments:

- If I type an email address, then hit tab to select a Role, I immediately
get an error saying that the role cannot be empty and the control loses
focus. I should be able to add a user using just the keyboard.

- In general I'm seeing validation errors before I've had a chance to enter
values. I should only see them if the appropriate field has lost focus.

- Role names should be "Administrator" and "User".

- The Close button should be disabled if errors are present.

- If I enter all the details for a new user and then hit Close, the dialog
is closed and the new user is NOT added. I have to click something else
first so the row loses focus, and then click close.

- The styling of the Add button and header does not match other grids (blue
background for the header, title in white on the left, grey "ADD" button
with no icon. The search box should be pushed right, to the left of the
button as well I think.

- s/Search by email/Filter by email

- s/New Password/New password

- s/Confirm Password/Confirm password

- The font on the Close button doesn't match what's on the properties
dialogues (looks like a global issue)

- The minimum size of the dialogue should be set such that the dialogue is
usable at mimimum size, e.g. all columns shown, with 2 rows.

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

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2016-06-03 12:10:53 pgAdmin 4 commit: Fix mouse cursor on dashboard quick links. Fixes #116
Previous Message Ashesh Vashi 2016-06-03 10:56:51 Re: Backgrid passwordCell [pgadmin4]