MIME-Version: 1.0
In-Reply-To: 
 <CAFiP3vz5y_sdiH6C5vH_zFY03vjEwwqXjy0LoViVyKudBpNBLQ@mail.gmail.com>
References: 
 <CAFiP3vz5y_sdiH6C5vH_zFY03vjEwwqXjy0LoViVyKudBpNBLQ@mail.gmail.com>
Date: Thu, 2 Jun 2016 09:29:56 +0100
Message-ID: 
 <CA+OCxox=d7AnK4BLz75BDSW2-ovXXMMhzbD8C=jYSWOa3iDDHw@mail.gmail.com>
Subject: Re: Regarding issue 1241
From: Dave Page <dpage@pgadmin.org>
To: Harshal Dhumal <harshal.dhumal@enterprisedb.com>
Cc: pgadmin-hackers <pgadmin-hackers@postgresql.org>
Content-Type: multipart/alternative; boundary=001a113fc002380a2d05344769c8
Precedence: bulk
Sender: pgadmin-hackers-owner@postgresql.org

--001a113fc002380a2d05344769c8
Content-Type: text/plain; charset=UTF-8

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

> Hi Dave,
>
> Regarding Issue 1241 <https://redmine.postgresql.org/issues/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

--001a113fc002380a2d05344769c8
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><br><div class=3D"gmail_extra"><br><div class=3D"gmail_quo=
te">On Tue, May 31, 2016 at 8:53 PM, Harshal Dhumal <span dir=3D"ltr">&lt;<=
a href=3D"mailto:harshal.dhumal@enterprisedb.com" target=3D"_blank">harshal=
.dhumal@enterprisedb.com</a>&gt;</span> wrote:<br><blockquote class=3D"gmai=
l_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left=
:1ex"><div dir=3D"ltr"><div>Hi Dave,<br></div><br>Regarding Issue <a href=
=3D"https://redmine.postgresql.org/issues/1241" target=3D"_blank">1241</a>:=
<br><div><br>We have added header section for parameter tab deliberately so=
 that we can force user to select parameter name (and therefore parameter&#=
39;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 sele=
cted in first cell (Name cell). See attached screen-shot.<br><br>For exampl=
e:<br>1. If user selects parameter &#39;array_nulls&#39; then value for thi=
s should be either true or false (and hence switch cell).<br>2. If user sel=
ects parameter &#39;cpu_index_tuple_cost&#39; then value for this should be=
 Integer (and hence Integer cell).<br><br>Without the custom header (and fo=
rcing user to select parameter) we cannot decide what type of cell we need =
in second column.<br><br>Let me know your opinion on this.<br></div></div><=
/blockquote><div><br></div><div>We need to figure out a way to fix it. Our =
difficulties encountered writing code should not dictate usability compromi=
ses. 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 p=
roblem, which is not appropriate as it leads to a poor look and feel and po=
tentially confusion for the user.</div><div><br></div></div><div><br></div>=
-- <br><div class=3D"gmail_signature" data-smartmail=3D"gmail_signature">Da=
ve Page<br>Blog: <a href=3D"http://pgsnake.blogspot.com" target=3D"_blank">=
http://pgsnake.blogspot.com</a><br>Twitter: @pgsnake<br><br>EnterpriseDB UK=
: <a href=3D"http://www.enterprisedb.com" target=3D"_blank">http://www.ente=
rprisedb.com</a><br>The Enterprise PostgreSQL Company<br></div>
</div></div>

--001a113fc002380a2d05344769c8--