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

From: Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
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 14:49:49
Message-ID: CAH4-4Wu6asogkiAjzZM_wvOXd53=YpfGb+qFmXd4pD7n_UpYwg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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.

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.zabuawala@
> enterprisedb.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
www.enterprisedb.com

Attachment Content-Type Size
current_new.png image/png 167.7 KB
subnode_current.png image/png 178.7 KB
subnode_new.png image/png 175.8 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-08-07 15:36:23 Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard
Previous Message Ashesh Vashi 2017-08-07 13:38:53 Re: [PATCH] Work with newer Flask-Security.