Re: [pgAdmin4][RM4232]: Change what is shown by default in tab titles - Mailing list pgadmin-hackers

From Nikhil Mohite
Subject Re: [pgAdmin4][RM4232]: Change what is shown by default in tab titles
Date
Msg-id CAOBg0AOHsocS1BObY6_umWc7VUdDhE0EFDzQv3b7Ng=Ui7EpQA@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][RM4232]: Change what is shown by default in tab titles  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Hi Akshay,

I have resolved the review comments and also added a %ARGS% placeholder for the debugger.
PFA patch v2.

Regards,
Nikhil Mohite.

On Mon, Oct 19, 2020 at 10:58 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Nikhil 

Following are the review comments:
  • Jasmin test cases are failing.
  • Can we change the following strings in preferences: 
    • "Query tool tab title placeholder" to "Query tool tab title"
    • "View/Edit tab title placeholder" to "View/Edit data tab title"
    • "Debugger tab title placeholder" to "Tab title" as this setting is already inside the Debugger section.
  • Change the help messages in the preferences dialog to:
    • "Supported placeholders are %DATABASE%, %USERNAME%, and %SERVER%. Users can provide any string with or without placeholders of their choice. The blank title will be revert back to the default title with placeholders." (Make the same changes for Query tool, View/Edit, and Debugger tab title message)
  • For Debugger can we add %ARGS% because in the case of functions with arguments tab title is very long, so the user should have the option to discard arguments in the tab title.
Update the documentation if you will make the above changes.

On Tue, Oct 13, 2020 at 5:27 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Team,

Regarding RM-4232 Change what is shown by default:
Added placeholders in preferences as following:
1. Debugger: Added placeholder in the display section of the debugger.
2. Query Tool and View/Edit data: Added placeholder in the display section of the Query Tool.

Updated the documentation for Query Tool and Debugger with new screenshots.

PFA patch.

--
Thanks & Regards,
Nikhil Mohite
Software Engineer.
Mob.No: +91-7798364578.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB Postgres
Mobile: +91 976-788-8246

Attachment

pgadmin-hackers by date:

Previous
From: Ganesh Jaybhay
Date:
Subject: Re: [pgAdmin][5919] Fix security related issues
Next
From: Akshay Joshi
Date:
Subject: Re: small change to Documentation - Creating a table