Re: [pgAdmin4][RM#3055] Allow user to sort the data in View data mode

From: Robert Eckhardt <reckhardt(at)pivotal(dot)io>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM#3055] Allow user to sort the data in View data mode
Date: 2018-03-27 13:36:57
Message-ID: CAAtBm9U=KZmKY--nA3o98T4Xg9+T8Ha0_hmcQzM8k36Dpu0-OQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, Mar 27, 2018 at 6:25 AM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> On Tue, Mar 27, 2018 at 3:13 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Mon, Mar 26, 2018 at 9:26 PM, Robert Eckhardt <reckhardt(at)pivotal(dot)io>
>> wrote:
>>
>>>
>>>
>>> On Mon, Mar 26, 2018 at 2:07 PM, Joao De Almeida Pereira <
>>> jdealmeidapereira(at)pivotal(dot)io> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> @Murtuza: The patch codewise looks good. Nice to see that we are using
>>>> axios instead of jquery ajax calls and that there is some coverage for the
>>>> change.
>>>> Nevertheless the Javascript testing looks a bit slim and could be
>>>> improved. Also the DataSorting class could have some other member functions
>>>> like the model validation could be extracted out so that it is easily
>>>> tested.
>>>>
>>>>
>>>> @Hackers: This was how we tried to test this feature:
>>>> 1 - Started pgAdmin
>>>> 2 - Opened the query tool for a specific server
>>>> 3 - Executed a SQL statment
>>>> 4 - Pressed the column header to try to order, nothing happened
>>>> 5 - Right clicked the column header to see if it was there the option,
>>>> nothing
>>>>
>>>> This is the behavior that we were expecting, not to have to open Data
>>>> View and then press the icon that is not even near the grid in order to
>>>> sort the column. Is this really the way we want people to use the grid in
>>>> pgAdmin? Should it be more intuitive?
>>>>
>>>
>>> Have we considered making the grid behave more like excel or other
>>> grids? I think that having the ascending and descending inside the column
>>> header, we could similarly provide filtering. Something that would give
>>> users a more intuitive place to look.
>>>
>>
>> Doing the sorting via header clicks is convenient but very restrictive.
>> How do you specify multiple columns to sort by for example? The current
>> design allows you to select columns and the sort order as you see fit.
>>
>
Honestly I'm not sold on my idea, I was just proposing an alternative in an
effort to start a discussion about the user experience. Ideally what I'd
like to see, maybe this happened, is some user research. When we initial
worked on refactoring the results grid we made a bunch of changes. One of
the things we intended to do was to follow up to see how people were using
the grid now so that we could better understand how it was now being used
in order to design and implement features just like this. Clearly we
haven't gotten there yet.

>
> Another reason we can't use that because w
> e have already occupied that behaviour for selecting entire column
> ​ when user clicks on header.
> As Dave suggested, I will be merging it with filter dialog meaning it will
> be accessible via direct button on toolbar & keyboard shortcut.​
> ​
>

How are users currently interacting with that filter dialog?

What I'm suggesting is that we understand how users want to interact with
their results, be those the results of a query or a table view, then we can
design something that meets those needs. I agree that changing the column
selection behavior isn't desirable, however, I also feel like providing the
best user experience is better than holding onto a particular feature
implementation.

-- Rob

>
>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2018-03-27 13:54:06 Re: [pgAdmin4][RM#3055] Allow user to sort the data in View data mode
Previous Message Joao De Almeida Pereira 2018-03-27 13:23:18 Re: [pgAdmin4][patch]: RM #3090 pgadmin shows misleading "Query returned successfully" with incorrect SQL