Re: Regarding issue 1241 - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Regarding issue 1241
Date
Msg-id CA+OCxox=d7AnK4BLz75BDSW2-ovXXMMhzbD8C=jYSWOa3iDDHw@mail.gmail.com
Whole thread Raw
In response to Regarding issue 1241  (Harshal Dhumal <harshal.dhumal@enterprisedb.com>)
Responses Re: Regarding issue 1241
List pgadmin-hackers


On Tue, May 31, 2016 at 8:53 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Dave,

Regarding Issue 1241:

We have added header section for parameter tab deliberately so that we can force user to select parameter name (and therefore parameter's data type) before adding new row. This is required because behavior of second cell (Value cell) is dependent on what parameter name user has selected in first cell (Name cell). See attached screen-shot.

For example:
1. If user selects parameter 'array_nulls' then value for this should be either true or false (and hence switch cell).
2. If user selects parameter 'cpu_index_tuple_cost' then value for this should be Integer (and hence Integer cell).

Without the custom header (and forcing user to select parameter) we cannot decide what type of cell we need in second column.

Let me know your opinion on this.

We need to figure out a way to fix it. Our difficulties encountered writing code should not dictate usability compromises. In this case, something that needs some thought and maybe some tricky code has caused us to create an inconsistent UI workflow to side-step the problem, which is not appropriate as it leads to a poor look and feel and potentially confusion for the user.


--
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: Susan Douglas
Date:
Subject: Patch for pgAdmin 4 docs
Next
From: Ashesh Vashi
Date:
Subject: Re: [pgAdmin4][Patch]: RM#1243 - Columns on the Query Tool should be sizeable