Обсуждение: [pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

Поиск
Список
Период
Сортировка

[pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

От
Chethana Kumar
Дата:
Hello Team,

I am sending a patch for the ex_index and ex_scan icons where those were similar to each other  so I have made them easy to differentiate now.

It is respective to the issue RM# 3021

Thanks,

Chethana Kumar
Principal UI/UX Designer
EnterpriseDB Corporation


The Postgres Database Company

P: +91 86981 57146
Вложения

Re: [pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

От
Dave Page
Дата:
Hi

On Thu, Jan 18, 2018 at 12:48 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hello Team,

I am sending a patch for the ex_index and ex_scan icons where those were similar to each other  so I have made them easy to differentiate now.

It is respective to the issue RM# 3021

They're still only distinguishable by colour, which isn't good for accessibility.  Can you think about an alternative design for index scans?

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Re: [pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

От
Chethana Kumar
Дата:
Hi Dave,

Can you please the icons as I have added a differentiating factor for them now.

Thanks,
Chethana kumar

On Mon, Jan 22, 2018 at 4:14 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Jan 18, 2018 at 12:48 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hello Team,

I am sending a patch for the ex_index and ex_scan icons where those were similar to each other  so I have made them easy to differentiate now.

It is respective to the issue RM# 3021

They're still only distinguishable by colour, which isn't good for accessibility.  Can you think about an alternative design for index scans?

Thanks.

--
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
Вложения

Re: [pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

От
Dave Page
Дата:
Hi

On Wed, Jan 24, 2018 at 7:47 AM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hi Dave,

Can you please the icons as I have added a differentiating factor for them now.

I think it's too detailed. If I view it at the typical 64x64 size, then I really can't make it out properly.

What about combining ex_index_only_scan with index scan? e.g. replace the lower tree nodes on index_only_scan with two or three table rows to indicate an index scan that then leads to a table access?
 

Thanks,
Chethana kumar

On Mon, Jan 22, 2018 at 4:14 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Jan 18, 2018 at 12:48 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hello Team,

I am sending a patch for the ex_index and ex_scan icons where those were similar to each other  so I have made them easy to differentiate now.

It is respective to the issue RM# 3021

They're still only distinguishable by colour, which isn't good for accessibility.  Can you think about an alternative design for index scans?

Thanks.

--
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




--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Re: [pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

От
Chethana Kumar
Дата:
Hi Dave,

As per your feedback here are the finalised icons attached.

Thanks,
Chethana kumar

On Wed, Jan 24, 2018 at 4:58 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Jan 24, 2018 at 7:47 AM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hi Dave,

Can you please the icons as I have added a differentiating factor for them now.

I think it's too detailed. If I view it at the typical 64x64 size, then I really can't make it out properly.

What about combining ex_index_only_scan with index scan? e.g. replace the lower tree nodes on index_only_scan with two or three table rows to indicate an index scan that then leads to a table access?
 

Thanks,
Chethana kumar

On Mon, Jan 22, 2018 at 4:14 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Jan 18, 2018 at 12:48 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hello Team,

I am sending a patch for the ex_index and ex_scan icons where those were similar to each other  so I have made them easy to differentiate now.

It is respective to the issue RM# 3021

They're still only distinguishable by colour, which isn't good for accessibility.  Can you think about an alternative design for index scans?

Thanks.

--
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




--
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
Вложения

Re: [pgAdmin4][Patch#3021] Updating existing Index scan & Scan icons

От
Dave Page
Дата:
Thanks, patch applied.

On Wed, Jan 24, 2018 at 1:40 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hi Dave,

As per your feedback here are the finalised icons attached.

Thanks,
Chethana kumar

On Wed, Jan 24, 2018 at 4:58 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Jan 24, 2018 at 7:47 AM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hi Dave,

Can you please the icons as I have added a differentiating factor for them now.

I think it's too detailed. If I view it at the typical 64x64 size, then I really can't make it out properly.

What about combining ex_index_only_scan with index scan? e.g. replace the lower tree nodes on index_only_scan with two or three table rows to indicate an index scan that then leads to a table access?
 

Thanks,
Chethana kumar

On Mon, Jan 22, 2018 at 4:14 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Jan 18, 2018 at 12:48 PM, Chethana Kumar <chethana.kumar@enterprisedb.com> wrote:
Hello Team,

I am sending a patch for the ex_index and ex_scan icons where those were similar to each other  so I have made them easy to differentiate now.

It is respective to the issue RM# 3021

They're still only distinguishable by colour, which isn't good for accessibility.  Can you think about an alternative design for index scans?

Thanks.

--
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




--
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




--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company