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

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Robert Eckhardt <reckhardt(at)pivotal(dot)io>
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:54:06
Message-ID: CAKKotZS+k_C3YGz3C=SUG9TRnr30zoS614BGJpPgzxjmCARoXQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, Mar 27, 2018 at 7:06 PM, Robert Eckhardt <reckhardt(at)pivotal(dot)io>
wrote:

>
>
> On Tue, Mar 27, 2018 at 6:25 AM, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.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?
>

​By clicking on the toolbar button as well as keyboard shortcut.

>
> 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 14:40:35 Re: pgadmin healthcheck-url
Previous Message Robert Eckhardt 2018-03-27 13:36:57 Re: [pgAdmin4][RM#3055] Allow user to sort the data in View data mode