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

Hi

On Mon, Jul 24, 2017 at 1:24 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Mon, Jul 24, 2017 at 8:51 AM, Shirley Wang <swang(at)pivotal(dot)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(dot)kumar(at)enterprisedb(dot)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
>