Re: [pgAdmin4][Patch]: Improvise the treeview element selection

Поиск
Список
Период
Сортировка
От Surinder Kumar
Тема Re: [pgAdmin4][Patch]: Improvise the treeview element selection
Дата
Msg-id CAM5-9D-vDBgwEExqdFUZSSZ=KRRrEXRLnR=ZMKBu-CBjeWkYfQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch]: Improvise the treeview element selection  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin4][Patch]: Improvise the treeview element selection  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Дерево обсуждения
Hi

On Mon, Jul 24, 2017 at 1:24 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Jul 24, 2017 at 8:51 AM, Shirley Wang <swang@pivotal.io> wrote:
Hi Surinder,

This change seems to be different from the other patterns for selected / highlighted options. Is there a reason why this needs to be more visible than it already is? In terms of feedback on the tree, we have not heard any pains from users that this is not visible enough.
​I agree it is visible enough but only with tree with 8 to 10 levels. If we think broader considering the case where function node may contain thousands of functions and then scrolling to previously selected function will become tougher.

Also, If we implement search by typing on selected node in future as in pgAdmin3, the selected node will be highlighted but finding selected node among other nodes will be painful due to lighter blue background..

Yeah, I'm inclined to agree. What's your motivation for this change? I've also never heard of any complaints about this.
​Two reasons:

1. If i select a function having long name from a list of 150 functions and then switch to query tool and then again i look for selected function, being light blue background it gets difficult to find selected function among 150 functions.

If background would be dark blue, it is easier to spot.

2. I liked the node selection in pgAdmin3 where selected node is highlighted with dark blue background. In pgAdmin4, only label of selected node is highlighted with light blue.

If this patch seems unreasonable, please ignore the patch. 
 

Shirley

On Mon, Jul 24, 2017 at 12:47 PM Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi All,

Changes:

1. Highlight the full selected tree item instead of highlighting only label.
2. Set selected item background color to dark blue that is more visible to eyes.[same as in pgAdmin3]

Please find attached patch and review.

Thanks,
Surinder



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

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

Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch]: Improvise the treeview element selection
Следующее
От: Harshal Dhumal
Дата:
Сообщение: [RM2579][pgAdmin4] Set default file listing layout as list instead of grid