From: | Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgAdmin 4 commit: Tree view icons |
Date: | 2017-09-22 12:24:49 |
Message-ID: | CAH4-4Wv+VOcq-=XSCd3qDD6VGo-t7=AE=LiiRBwdB0+4ZYP_bQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave,
PFA patch with resolved all the issues raised.
Issues:
1. Alignment issue: All icons changed from left to center aligned
2. Missing job step icon: Changed in css from .png to .svg for loading the
icons
3. Similar look between Jobs and job icon: Updated the icon to
differentiate themselves
Also added a white rounded box for the icon placed on popup title bar to
differentiate the icon blue color from blue title bar.
Regards,
Chethana kumar
Regards,
Chethana kumar
On Fri, Sep 22, 2017 at 4:06 PM, Chethana Kumar <
chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
>
> On Fri, Sep 22, 2017 at 3:06 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi,
>>
>> Couple of issues I spotted in testing this:
>>
>> - pgAgent Job Steps no longer display an icon.
>>
>
> It is because of the file naming convention which is not matching with the
> css call.
> We need to ensure it once.
>
>
>>
>> - pgAdmin jobs and job look like they share the same icon - which is
>> quite off-center.
>>
>> I have resolved this and share the patch.
>
>
>> Screenshot attached.
>>
>> On Thu, Sep 21, 2017 at 11:29 AM, Chethana Kumar <
>> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> PFA patch which resolves the alignment issues of the tree view icons.
>>>
>>> Regards,
>>>
>>> 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
>>
>
>
>
> --
> 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
Attachment | Content-Type | Size |
---|---|---|
treeview_icons_issues_resolved_22_SEP.patch | application/octet-stream | 314.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Raphaël ENRICI | 2017-09-22 12:43:46 | Re: pgAdmin 4 debian packaging (?) |
Previous Message | Devrim Gündüz | 2017-09-22 12:24:24 | Re: pgAdmin 4 debian packaging (?) |