Re: [pgAdmin][RM3782] Debugger title should show connection and object details

From: "Strauch, Sheldon" <sstrauch(at)enova(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM3782] Debugger title should show connection and object details
Date: 2019-06-12 15:26:23
Message-ID: CAONmzmVGa5o5aRMAqELVcvfrB3TK68uWrvVSq3-Qao_3iE3wNA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

FWIW:

- +1 for INDEPENDENTLY configurable tab title AND tooltip
- I agree that in-place edit of tabs cost/benefit isn't there, unless
someone wants a personal free-time project

On Wed, Jun 12, 2019 at 4:44 AM Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Wed, Jun 12, 2019 at 10:02 AM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> On Wed, Jun 12, 2019 at 1:54 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Wed, Jun 12, 2019 at 7:03 AM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Attached is the patch to add more information on connection details to
>>>> debugger panel title and content title similar to Query Tool and View/Edit
>>>> data.
>>>> Adding the object name with arguments is not a good idea for the panel
>>>> title, so I have kept to connection details similar to query tool. The
>>>> content title will have all the details.
>>>>
>>>> Kindly review.
>>>>
>>>
>>> I think not having the function name in there is a big limitation.
>>>
>>> With all this hacking going on, I think what we need to do is stop and
>>> make this configurable as has been both requested by users and suggested by
>>> us.
>>>
>>> I propose adding 2 preference options for each tab type. Each of these
>>> would be a string with placeholders, one for the tab title, and one for the
>>> tab tooltip. For example:
>>>
>>> - Query Tool
>>> * Placeholders: %DATABASE%, %SERVER%, %USERNAME%, %PORT%, %HOST%
>>> * Tab title default: %DATABASE% on %SERVER%
>>> * Tooltip title default: Query Tool: %DATABASE% on %USERNAME%(at)%SERVER%
>>>
>>> - View Data
>>> * Placeholders: %TABLE%, %SCHEMA%, %DATABASE%, %SERVER%, %USERNAME%,
>>> %PORT%, %HOST%
>>> * Tab title default: %SCHEMA%.%TABLE% in %DATABASE% on %SERVER%
>>> * Tooltip title default: View/Edit Data: %SCHEMA%.%TABLE%
>>> in %DATABASE% on %USERNAME%(at)%SERVER%
>>>
>>> - Script
>>> * Placeholders: %ACTION%, %OBJECT%, %SCHEMA%, %DATABASE%, %SERVER%,
>>> %USERNAME%, %PORT%, %HOST%
>>> * Tab title default: %ACTION% %SCHEMA%.%OBJECT% in %DATABASE% on
>>> %SERVER%
>>> * Tooltip title default: %ACTION% Script: %SCHEMA%.%OBJECT%
>>> in %DATABASE% on %USERNAME%(at)%SERVER%
>>>
>>> - Debugger
>>> * Placeholders: %FUNCTION_ARGS%, %FUNCTION_NOARGS%, %SCHEMA%,
>>> %DATABASE%, %SERVER%, %USERNAME%, %PORT%, %HOST%
>>> * Tab title default: %FUNCTION_NOARGS%.%OBJECT% in %DATABASE% on
>>> %SERVER%
>>> * Tooltip title default: Debugger: %SCHEMA%.%FUNCTION_ARGS%
>>> in %DATABASE% on %USERNAME%(at)%SERVER%
>>>
>>> Or something like that. Thoughts?
>>>
>> Yes, this could be a nice feature. I think tooltip title customisation is
>> not needed, we can just set it as prefix + tab title.
>>
>
> I think having it be separate is useful - you can include full information
> there, but keep the tab minimal so as not to create huge tabs. For example,
> the debugger tab could just be set to %FUNCTION_NOARGS%, whilst the tooltip
> has the full string.
>
>
>> Some users asked for in-place rename of tab title. I'm not sure if it is
>> feasible to implement, but should we consider it also ?
>>
>
> No - I've only seen one request for that, and it seems like a lot of work
> for little gain.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> <https://urldefense.proofpoint.com/v2/url?u=http-3A__pgsnake.blogspot.com&d=DwMFaQ&c=lEzKI_JJakPtcnbAQ6Q5xQ&r=3P3-5UqoQFsuwPHL1Gr-FtP4nR5GVvvXlP89XkF_Y4w&m=m7F0ZO6XISz4efeKAoiH-zgc28fGDF5D28EPqWoR8Zc&s=Se3Rf1mZ5YUUpEeSpju_2WtOAmGZQV5utDkHjTpbfSg&e=>
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> <https://urldefense.proofpoint.com/v2/url?u=http-3A__www.enterprisedb.com&d=DwMFaQ&c=lEzKI_JJakPtcnbAQ6Q5xQ&r=3P3-5UqoQFsuwPHL1Gr-FtP4nR5GVvvXlP89XkF_Y4w&m=m7F0ZO6XISz4efeKAoiH-zgc28fGDF5D28EPqWoR8Zc&s=a8Bm7sUnUKywflPSQ98o2ha5k8oBtacucx-pjrIXTIc&e=>
> The Enterprise PostgreSQL Company
>

--

Look after your data, and your database will look after you. -- Simon Riggs

Sheldon E. Strauch
*Data Architect, Data Services *
*O* 312-676-1556
*M* 224-723-3878

*Enova International, Inc.*
*This transmission is confidential and may be privileged or proprietary. If
you are not the intended recipient, you are not authorized to use the
information in this transmission in any way. Please inform the sender
immediately if you have received this transmission in error and permanently
delete and destroy the original and any copies of the information.*

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Aditya Toshniwal 2019-06-13 05:06:11 Re: [pgAdmin][RM3782] Debugger title should show connection and object details
Previous Message Dave Page 2019-06-12 12:52:31 Re: [RM#4350][pgAdmin4] User is not able to upload files using File manager.