From: | Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, Khushboo Vashi <khushboo(dot)vashi(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:23:49 |
Message-ID: | CAH4-4WvZ1QU9gCq-JCTOWK_LsrEOAZdEaS5zqub-+finCfBF-Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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]
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*
>
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-09-01 06:27:19 | Re: Macro Support UI Design |
Previous Message | Akshay Joshi | 2020-08-31 12:23:02 | Re: [pgAdmin][Patch] #5746 setup.py --load-server does not allow loading connections that use pg_services |