From: | Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>, Dave Page <dpage(at)pgadmin(dot)org>, 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-09-11 09:34:58 |
Message-ID: | CAH4-4WuuNriXD_UYExuLQNH93n8wL+FiiEHromnZEw5cjHoYdQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Mon, Sep 11, 2017 at 1:15 PM, Murtuza Zabuawala <murtuza.zabuawala@
enterprisedb.com> wrote:
> Hello,
>
> I also observed few issues,
>
> 1) In my opinion, New icons looks dull in main header menu due to dark
> background, attaching screenshot for your reference.
>
Icons for the dialog boxes area will go in white.
>
> 2) Spacing issue between icons and label are inconsistent, attaching
> screenshot for your reference.
>
>
Working on it
> 3) Icons with blue colour in them are not properly visible in context
> menu. For example "Trigger" icon is not visible when we hover over it
> because of same colour, attaching screenshot for your reference.
>
>
Same as 1st one
> 4) Are we considering different icons for Event triggers & Triggers? (They
> were same in pgAdmin3)
>
Yes we are keeping them different so that user can easily differentiate
them
> 5) Icons for Casts modules are same, I mean collection icon and node icon,
> I think we need different icons to differentiate.
>
Its agreed and done the updates, attached the new icon
> 6) The width of bad-server-icon is more when compare with other icons,
> that makes it looks like a text document icon.
>
if you check the measurement both are same, well I have done some tweaking
to it now. Attached them for review.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> [image: https://community.postgresrocks.net/]
> <https://community.postgresrocks.net/>
>
> On Mon, Sep 11, 2017 at 12:58 PM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Dave,
>>
>> As far as I know, we replaced png icons with svg icons because - it
>> should work with all types of resolutions. The svg icon should not blur.
>> Here in my Linux system, I am using 1440*900(16:10) resolution and most
>> of the svg icons are blurred. Please find attach screenshot.
>>
>> Thoughts ?
>>
>> Thanks,
>> Neel Patel
>>
>> On Fri, Aug 25, 2017 at 5:14 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> 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(dot)zabuawala(at)enterprisedb(dot)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
>>>
>>
>>
>
--
Chethana Kumar
Principal UI/UX Designer
EnterpriseDB Corporation
The Postgres Database Company
P: +91 86981 57146
www.enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
server.svg | image/svg+xml | 858 bytes |
serverbad.svg | image/svg+xml | 1.0 KB |
coll-cast.svg | image/svg+xml | 2.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | pgAdmin 4 Jenkins | 2017-09-11 09:36:46 | Build failed in Jenkins: pgadmin4-master-python26 #438 |
Previous Message | Dave Page | 2017-09-11 09:31:09 | Re: Build failed in Jenkins: pgadmin4-master-python26 #436 |