Re: pgAdmin4 - Tree view icons for final reveiw

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Anthony DeBarros <adebarros(at)gmail(dot)com>
Cc: Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com>, Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>, Shirley Wang <swang(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin4 - Tree view icons for final reveiw
Date: 2017-08-25 11:44:37
Message-ID: CA+OCxozEFy3qQWGi2GS2GjJcnmuutX7A+Cs7KjOwBrrVRUM6ZQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

Yes, that's my expectation. Hopefully they'll go in in the next week or so.

On Fri, Aug 25, 2017 at 12:40 PM, Anthony DeBarros <adebarros(at)gmail(dot)com>
wrote:

> Dave and all,
>
> Out of curiosity (because I’m writing a book that includes pgAdmin screen
> grabs), do you expect these icons to make it into your September release?
>
> Thanks,
> Anthony
>
>
> On August 23, 2017 at 4:03:42 AM, Murtuza Zabuawala (murtuza.zabuawala@
> enterprisedb.com) wrote:
>
> Hi Chethana,
>
>
> On Wed, Aug 23, 2017 at 12:51 PM, Chethana Kumar <
> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>
>> Hi Murtuza,
>>
>> Please see my reply inline along with the updated .svg file for tree view
>> icons.
>>
>>
>> On Tue, Aug 22, 2017 at 11:17 AM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Chethana,
>>>
>>> - The icons for Server node is missing, we will need given set of icons
>>> for server.
>>> 1) Basic Server icon (Disconnect state)
>>> 2) PG (Connected state)
>>> 3) EPAS (Connected state)
>>> 4) GreenPlum (Connected state)
>>> [Attaching current icons for PG & EPAS, I'm not sure about GreenPlum
>>> server icon.]
>>>
>>> - I'm not sure where we will use serverbad icon from
>>> 'tree_view_Icons.svg' file as it looks like server-group icon and not
>>> individual server.
>>>
>> The icon is there in the pgAdmin's icon folder and it is needed for the
>> application
>>
> ​Yes, we need serverbad icon which indicates the server in disconnect
> state but what we have on svg is same icon as server-group icon, we need
> icon for individual server.​
>
> [image: Inline image 1]
>
> ​Something like this below which will differentiate individual server icon
> from server-group icon,
> [image: Inline image 6]​
>
>
>>> - Icon set for Schema & Catalog (both collection and individual) looks
>>> very similar to each other with minor colour difference, we have to look
>>> very carefully to identify them each.
>>>
>> Agree with this point and come up with some updations on it. This time
>> I have made it pretty differentiable between Schema and Catalog.
>>
>>>
>>> - Why do we require two different set of icons for Tables & Columns, I
>>> mean one for Schema and one for Catalog?
>>>
>> Agreed, I have kept single icon for tables and columns.
>>
>>>
>>> - Icons for Partition node is same for collection and individual.
>>>
>> Working on it
>>
>>>
>>> - Icons for Event trigger node and Trigger node are very different from
>>> each other despite both are triggers.
>>>
>> Agreed, done the changes accordingly.
>>
> ​This is still different.
>
> *Current Event trigger icon:*
> [image: Inline image 4]
> *New*
> *​ *
> *Event trigger*
> *​*
> *Icon:*
> [image: Inline image 2]​
>
>
> *Current Trigger icon:*
>
> *[image: Inline image 5]*
> *New Trigger Icon:*
> [image: Inline image 3]
> ​If you see both icons are same.
> *[Suggestion: Can we do similar to what we have done with View &
> Materialized view icons, we have added M in the icon for materialized view,
> Same way we can have Trigger icon and for Event trigger we can add E in the
> icon]*
>
>
>> I know this is out of context but in my own opinion these icons have
>>> resemblance with Flat design where as rest of our application is in 2d
>>> design, for example buttons, dialogs etc.
>>>
>> Yes, the icons look and feel are more towards flat and stylish now as it
>> represents the current trend. You treat this as a first step towards
>> making the whole application flat design from the current design. But
>> this will happen in a long run, not on a quick basis.
>>
>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>>
>>>
>>> On Tue, Aug 22, 2017 at 9:37 AM, Chethana Kumar <
>>> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>>>
>>>> Yes Shirley, I have already thought through on it.
>>>> Basically, we need to have one more set of icons in white color mode so
>>>> that we can use it on dialog headers as well.
>>>> As the blue theme icons won't fit there.
>>>>
>>>> Please share your thoughts.
>>>>
>>>> Regards,
>>>> Chethana kumar
>>>>
>>>> On Tue, Aug 22, 2017 at 4:32 AM, Shirley Wang <swang(at)pivotal(dot)io> wrote:
>>>>
>>>>> Hi Chethana,
>>>>>
>>>>> I just thought of something with these icons. What happens with the
>>>>> ones that appear in dialog headers? In the blue bar?
>>>>>
>>>>> On Mon, Aug 21, 2017 at 8:31 AM Chethana Kumar <
>>>>> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hello Team,
>>>>>>
>>>>>> I have applied labels for each icon for identification purpose. So
>>>>>> please consider this copy for your feedback.
>>>>>>
>>>>>> *Note*: The "icon_demo.png" is attached to show case the view on
>>>>>> webpage.
>>>>>>
>>>>>> Regards,
>>>>>> Chethana kumar
>>>>>>
>>>>>> On Mon, Aug 21, 2017 at 3:57 PM, Chethana Kumar <
>>>>>> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>>>>>>
>>>>>>> Hello Team,
>>>>>>>
>>>>>>> Here are the revised and final icons for tree view control.
>>>>>>> I have attached in .svg file format as well so that you can do zoom
>>>>>>> in view at any level.
>>>>>>>
>>>>>>> Please share your feedback on the same.
>>>>>>>
>>>>>>>
>>>>>>> Thanks and regards,
>>>>>>>
>>>>>>> Chethana Kumar
>>>>>>> Principal UI/UX Designer
>>>>>>> EnterpriseDB Corporation
>>>>>>>
>>>>>>>
>>>>>>> The Postgres Database Company
>>>>>>>
>>>>>>> P: +91 86981 57146 <+91%2086981%2057146>
>>>>>>> www.enterprisedb.com
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Chethana Kumar
>>>>>> Principal UI/UX Designer
>>>>>> EnterpriseDB Corporation
>>>>>>
>>>>>>
>>>>>> The Postgres Database Company
>>>>>>
>>>>>> P: +91 86981 57146 <+91%2086981%2057146>
>>>>>> www.enterprisedb.com
>>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Chethana Kumar
>>>> Principal UI/UX Designer
>>>> EnterpriseDB Corporation
>>>>
>>>>
>>>> The Postgres Database Company
>>>>
>>>> P: +91 86981 57146
>>>> www.enterprisedb.com
>>>>
>>>
>>>
>>
>>
>> --
>> Chethana Kumar
>> Principal UI/UX Designer
>> EnterpriseDB Corporation
>>
>>
>> The Postgres Database Company
>>
>> P: +91 86981 57146
>> 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 Ashesh Vashi 2017-08-25 11:46:50 pgAdmin 4 commit: Fix the feature tests failuers.
Previous Message Anthony DeBarros 2017-08-25 11:40:28 Re: pgAdmin4 - Tree view icons for final reveiw