Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. - Mailing list pgadmin-hackers

From Dave Page
Subject Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion.
Date
Msg-id CA+OCxoyH50_sGqPKv9GL3GRFfQ5=u2LRTJa7gh=YPKyGqCS0Qg@mail.gmail.com
Whole thread Raw
In response to RM-4232- Change what is shown by default in tab titles: Implementation suggestion.  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
Responses Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion.  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
List pgadmin-hackers
Hi

On Thu, Oct 8, 2020 at 6:48 AM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Team,

Regarding feature RM-4232, Change what is shown by default in tab titles, 

I had a discussion with Akshay based on that I have added new options in preferences for the query tool/View-Edit and Debugger.
1. Query tool and View/Edit:
Added 2 options (placeholders) under display settings of Query Tool.
image.png
default it is set to what we are currently using, If the user updates the placeholders we will show the tab titles accordingly.

OK.
 

2. Debugger:
Added tab title place holder in display settings of the debugger.
image.png
default we are showing only "Debugger" as tab title.

1. If the user added any string with a placeholder that string will get a display in the tab title along with a placeholder.
2. Tooltip for the tabs will be the same as the placeholders that the user provided in the preferences.

Currently the tooltips are different iirc, as they sometimes have longer strings. Maybe we should allow them to be customised in the same way?
 

Questions:
1. For scripts we are showing the same title as the Query Tool, do we need a different placeholder for it or we can reuse the Query Tool placeholder for it? 
    (If we need to add a separate placeholder for scripts, where will be the right place to put it in the preferences?)

No, scripts are just an entry point to the query tool.
 
2. Do we need to provide a placeholder for Debugger, as we are showing only "Debugger" in the title.

I think we should - and that does seem like one case where we should change the default to be the function name or similar.
 

Any suggestions or anything I have missed in this please let me know.

The Help text on the Preferences should list the placeholders the user can use (I don't think there's any point in stating what the default is). I would also suggest that setting the value to an empty string should cause it to return to the default (so when the preferences are reopened, the default value is shown, and not an empty string).
 
Thanks!

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: http://www.enterprisedb.com

Attachment

pgadmin-hackers by date:

Previous
From: Aditya Toshniwal
Date:
Subject: [pgAdmin][patch] Packages security fixes
Next
From: Nikhil Mohite
Date:
Subject: Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion.