Re: Macro Support UI Design

From: Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, Dave Page <dave(dot)page(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Macro Support UI Design
Date: 2020-09-01 06:36:23
Message-ID: CAH4-4WsT86QtW++G1WQpAcJMm6CPBkHYWrRT7neDBLiUKYpqMg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, Sep 1, 2020 at 11:59 AM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

>
>
> On Tue, Sep 1, 2020 at 11:54 AM Chethana Kumar <
> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>
>> Hi Team,
>>
>> I am sort of ok with Khushboo's design solution which re-uses the
>> existing UI element called sub-node control.
>>
>> At the same time, I wanted to increase the ease of use and less cluttered
>> there.
>> So for that, I had to get rid of the subnode control to come up with the
>> new thoughts on to it.
>>
>> I have come up with 2 samples where they have a different way of *clearing
>> multiple shortcut keys.*
>>
>> Sample1-
>> [image: sample1.jpg]
>>
>> I like this design (Sample1)
>

Yes me too because the interaction pattern is as same as the "checkbox" one
that we have seen already.
It just inherits the interaction pattern of it.

But let see what others feel.

---
Chethana Kumar

> Sample2
>> [image: sample2.jpg]
>>
>> *Annotation-*
>>
>> [image: manage_macro.png]
>>
>>
>> *Functionality Animation -*
>> [image: manage_macro.gif]
>>
>> I have also attached icon for '*macro*' and I have a small correction
>> with the dropdown menu as shown below -
>>
>> [image: option.png]
>>
>> Correction as *Manage macro... *in place of *Manage Macro*
>>
>> I need your comments on the same.
>>
>>
>> ----
>> Chethana Kumar
>>
>>
>>
>> On Thu, Aug 27, 2020 at 6:55 PM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Chethana
>>>
>>> On Thu, Aug 27, 2020 at 6:27 PM Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Chethana,
>>>>
>>>> On Thu, Aug 27, 2020 at 5:58 PM Chethana Kumar <
>>>> chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Khushboo,
>>>>>
>>>>> I have a couple of opinions -
>>>>>
>>>>> - Can we place "Name" in the first position and then "Key" at the
>>>>> second?
>>>>>
>>>> Basically the key is going to be the non-editable field (unique key)
>>>> and the user can not change it. Users can only change name and SQL, so the
>>>> key is in first place.
>>>>
>>>>> - I think 'delete' icon suites more than 'clear icon'
>>>>>
>>>> We use the delete icon to completely delete the row from the grid but
>>>> here we are not going to delete the row, only clearing the name and SQL
>>>> fields for the selected row. So, the macro details against that key will be
>>>> removed from the database but the key remains in the database.
>>>>
>>>
>>> For the above two points, I agree with Khushboo.
>>>
>>> - Can we have the dropdown menu as below? As you can see user can select
>>>>> the "entered query from the query editor" then directly press "New Macro"
>>>>> which saves the user time and energy too. Please correct me if this is not
>>>>> something logical working one.
>>>>>
>>>> Sure, we can have a drop down menu as you suggested but not sure about
>>>> the selecting the entered query and can use that directly in the New Macro
>>>> dialog as we will have a list of macros.
>>>>
>>>
>>> I will go with "Manage macros..." instead of "<New Macro>" as it
>>> opens the dialog where the user can edit the Query and Name, we are not
>>> adding any new keyboard shortcuts. Refer to the pgAdmin 3 screenshot.
>>> [image: Screenshot 2020-08-27 at 6.38.47 PM.png]
>>>
>>> Regarding select the query from the query editor and click on the
>>> menu option won't solve the purpose as we don't know which shortcut key to
>>> map this query.
>>>
>>>>
>>>> Thanks,
>>>> Khushboo
>>>>
>>>>>
>>>>> [image: macro.png]
>>>>>
>>>>> ---
>>>>> Chethana Kumar
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> On Thu, Aug 27, 2020 at 5:37 PM Khushboo Vashi <
>>>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>>
>>>>>>
>>>>>> On Thu, Aug 27, 2020 at 5:34 PM Dave Page <dave(dot)page(at)enterprisedb(dot)com>
>>>>>> wrote:
>>>>>>
>>>>>>> I think you sent this to the wrong mailing list.
>>>>>>>
>>>>>> Forwarded to pgadmin-hackers. :)
>>>>>>
>>>>>>>
>>>>>>> On Thu, Aug 27, 2020 at 11:08 AM Khushboo Vashi <
>>>>>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>>>>>
>>>>>>>> Hi Team,
>>>>>>>>
>>>>>>>> I have started working on the feature #1402: Macro support like
>>>>>>>> pgAdmin 3.
>>>>>>>>
>>>>>>>> I am thinking of the design below.
>>>>>>>>
>>>>>>>> [image: Macro.png]
>>>>>>>>
>>>>>>>> - On clicking upon the Manage Macro, the dialogue will be opened
>>>>>>>> - In the grid, I have provided Clear Icon instead of Delete as we
>>>>>>>> will have all the supported macro listed with the shortcut key and the key
>>>>>>>> will not be editable, so no need to delete the row.
>>>>>>>> - Add button will be disabled or can be removed.
>>>>>>>>
>>>>>>>> - All the configured macros by the user will be displayed in the
>>>>>>>> list below the Manage Macros menu item and on clicking on the particular
>>>>>>>> macro, it will be executed in the SQL Editor
>>>>>>>>
>>>>>>>> Please provide feedback/suggestions.
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Khushboo
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Dave Page
>>>>>>> VP & Chief Architect, Database Infrastructure
>>>>>>> EDB: http://www.enterprisedb.com
>>>>>>>
>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>> Twitter: @pgsnake
>>>>>>>
>>>>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>> *pgAdmin Hacker | Sr. Software Architect*
>>> *EDB Postgres <http://edbpostgres.com>*
>>>
>>> *Mobile: +91 976-788-8246*
>>>
>>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2020-09-01 07:32:09 pgAdmin 4 commit: Fixed accessibility issues for main panels.
Previous Message Ganesh Jaybhay 2020-09-01 06:34:23 [pgAdmin] Accessibility fixes for main panels