Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com>
Cc: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>, Shirley Wang <swang(at)pivotal(dot)io>, Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard
Date: 2017-08-07 15:36:23
Message-ID: CA+OCxozRsmAafTDeZ+=vK+eztuh4+_stvXxtq+CRCokfjnz3KQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Mon, Aug 7, 2017 at 3:49 PM, Chethana Kumar <
chethana(dot)kumar(at)enterprisedb(dot)com> wrote:

> Hi Dave,
>
> Could you comment on the new design update for subnode control ?
>
> Attached designs -
>
> 1. subnode_current.png - The existing view
> 2. subnode_new.png - The updated view
> 3. current_new.png - Existing and new design placed together for
> comparison purpose
>
> Below changes done -
>
> 1. Reduced number of gray color variations
> 2. Made more simplified by removing unwanted borders
> 3. Merged "Arrow Down" background and the body background for more clarity
>
> Please feel free to provide your input on the same.
>

Much nicer in my opinion! Good work.

Anyone else have any comments?

>
> Regards,
> Chethana kumar
>
>
>
> On Fri, Jul 28, 2017 at 4:11 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> I took a quick look at this and have a couple of thoughts:
>>
>> - Instead of the "edit" icon to open the subnode, we should use something
>> more appropriate - a "properties" icon perhaps.
>>
>> - There seems to be a lot of different shades of grey on there (maybe a
>> subnode design in general that just shows up with the disabled controls),
>> and the subnode control looks a bit messy as a result.
>>
>> Can you work with Chethana to improve the look and feel please?
>>
>> Input from others welcome of course - screenshot attached.
>>
>> Thanks.
>>
>> On Fri, Jul 28, 2017 at 11:33 AM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> ++ Attaching the patch
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Fri, Jul 28, 2017 at 4:02 PM, Murtuza Zabuawala <
>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Dave,
>>>>
>>>> PFA patch to display additional information from pg_stat_activity table
>>>> using subnode control.
>>>> RM#2597
>>>>
>>>> Please review.
>>>>
>>>> --
>>>> Regards,
>>>> Murtuza Zabuawala
>>>> EnterpriseDB: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>> On Tue, Jul 25, 2017 at 10:56 AM, Shirley Wang <swang(at)pivotal(dot)io>
>>>> wrote:
>>>>
>>>>>
>>>>>> On Mon, Jul 24, 2017 at 8:11 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Mon, Jul 24, 2017 at 3:28 PM, Shirley Wang <swang(at)pivotal(dot)io>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> 2-3 days is a lot of valuable engineering time. Is this a 'drop
>>>>>>>> everything now' kind of feature or can this wait for some user validation
>>>>>>>> on a mock up first?
>>>>>>>>
>>>>>>>
>>>>>>> Most of the time will likely be on the infrastructure to change the
>>>>>>> display to a subnode control. If you have some cycles to mockup potential
>>>>>>> layouts for the subnode view and have them validated, please feel free,
>>>>>>> however, that seems like an awful lot of work to me to display some missing
>>>>>>> SQL using a standard control.
>>>>>>>
>>>>>> Regarding SQL display: Developing simple control to show codemirror
>>>>>> in disabled state (for now) wont take that much time.
>>>>>>
>>>>>>
>>>>> Part of a product designer's job is to make sure there is a definitive
>>>>> need for a feature and that the interface for the feature is designed in
>>>>> such a way that the user gets all intended value from it. Time spent
>>>>> validating now will decrease the time spent later on redesigning /
>>>>> reimplementing.
>>>>>
>>>>> If everyone is aware of what that value is and confident that how
>>>>> it'll be displayed is right, there's little risk in starting to develop it.
>>>>> If we're wrong, it'll add to feature bloat and detract from the experience.
>>>>>
>>>>> Would Chethana be able to take on some of the design work? It would be
>>>>> valuable for the dev team to also be part of design process.
>>>>>
>>>>
>>>>
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
>
> --
> Chethana Kumar
> Principal UI/UX Designer
> EnterpriseDB Corporation
>
>
> The Postgres Database Company
>
> P: +91 86981 57146 <+91%2086981%2057146>
> www.enterprisedb.com
>

--
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 Cramer 2017-08-07 18:53:54 Fix key error
Previous Message Chethana Kumar 2017-08-07 14:49:49 Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard