From: | Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [PgAdmin][RM-4231]: Dynamically size tab titles. |
Date: | 2020-11-04 07:13:45 |
Message-ID: | CAOBg0AN1MHKmKNy1U5hXSwgsV018+Ymkgyhr=z_UPm0P6bgRXA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
Please find the updated patch, resolved all review comments.
Regards,
Nikhil Mohite.
On Wed, Nov 4, 2020 at 11:24 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Nikhil
>
> Following are the review comments:
>
> - "Tabs settings" should be renamed to "Tab settings".
> - "Debugger Tab title" should be renamed to the "Debugger tab title".
> - "Enable dynamic tab size" should be renamed to "Dynamic tab size".
> - Add help message for the "Dynamic tab size" option mentioning that
> it will affect the existing opened tab.
> - The size of switch control <True/False> needs to be increased, as
> the text is not properly displayed.
> - Alignment is completely changed for controls under "Keyboard
> shortcuts" options.
>
> Please update the documentation as well after above changes.
>
> On Mon, Nov 2, 2020 at 1:55 PM Nikhil Mohite <
> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>
>> Hi Team,
>>
>> Please find the patch for RM-4231
>> <https://redmine.postgresql.org/issues/4231>: Dynamically size tab
>> titles.
>> 1. Added "Tab settings" in browser preferences.
>> 2. Added setting to enable dynamic tab size in Tab settings.
>> 3. Moved the following tab related setting into Tab setting.
>>
>> - Open in a new browser tab: For query tool, debugger, and schema
>> diff.
>> - Placeholders for Query tool, View/Edit data, and Debugger.
>>
>> 4. Adde migration to remove older "open new tab browser tab" preferences
>> from the database.
>> 5. Updated the document as per new changes.
>>
>> Added small CSS fix for RM-5950
>> <https://redmine.postgresql.org/issues/5950>: Long backup file name not
>> showing fully on process watcher dialog.
>>
>>
>>
>> --
>> *Thanks & Regards,*
>> *Nikhil Mohite*
>> *Software Engineer.*
>> *EDB Postgres* <https://www.enterprisedb.com/>
>> *Mob.No: +91-7798364578.*
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
Attachment | Content-Type | Size |
---|---|---|
RM_4231_v3.patch | application/octet-stream | 2.8 MB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-11-04 12:16:15 | pgAdmin 4 commit: 1) Added support for dynamic tab size. Fixes #4231 |
Previous Message | Akshay Joshi | 2020-11-04 05:54:31 | Re: [PgAdmin][RM-4231]: Dynamically size tab titles. |