public inbox for [email protected]
help / color / mirror / Atom feedRM-4232- Change what is shown by default in tab titles: Implementation suggestion.
4+ messages / 2 participants
[nested] [flat]
* RM-4232- Change what is shown by default in tab titles: Implementation suggestion.
@ 2020-10-08 05:47 Nikhil Mohite <[email protected]>
2020-10-08 08:41 ` Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Dave Page <[email protected]>
0 siblings, 1 reply; 4+ messages in thread
From: Nikhil Mohite @ 2020-10-08 05:47 UTC (permalink / raw)
To: pgadmin-hackers
Hi Team,
Regarding feature RM-4232 <https://redmine.postgresql.org/issues/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: 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.
2. Debugger:
Added tab title place holder in display settings of the debugger.
[image: 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.
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?)
2. Do we need to provide a placeholder for Debugger, as we are showing only
"Debugger" in the title.
Any suggestions or anything I have missed in this please let me know.
--
*Thanks & Regards,*
*Nikhil Mohite*
*Software Engineer.*
*EDB Postgres* <https://www.enterprisedb.com/;
*Mob.No: +91-7798364578.*
Attachments:
[image/png] image.png (288.7K, 3-image.png)
download | view image
[image/png] image.png (157.6K, 4-image.png)
download | view image
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion.
2020-10-08 05:47 RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Nikhil Mohite <[email protected]>
@ 2020-10-08 08:41 ` Dave Page <[email protected]>
2020-10-08 09:34 ` Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Nikhil Mohite <[email protected]>
0 siblings, 1 reply; 4+ messages in thread
From: Dave Page @ 2020-10-08 08:41 UTC (permalink / raw)
To: Nikhil Mohite <[email protected]>; +Cc: pgadmin-hackers
Hi
On Thu, Oct 8, 2020 at 6:48 AM Nikhil Mohite <[email protected]>
wrote:
> Hi Team,
>
> Regarding feature RM-4232 <https://redmine.postgresql.org/issues/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: 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: 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
Attachments:
[image/png] image.png (288.7K, 3-image.png)
download | view image
[image/png] image.png (157.6K, 4-image.png)
download | view image
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion.
2020-10-08 05:47 RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Nikhil Mohite <[email protected]>
2020-10-08 08:41 ` Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Dave Page <[email protected]>
@ 2020-10-08 09:34 ` Nikhil Mohite <[email protected]>
2020-10-08 09:39 ` Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Dave Page <[email protected]>
0 siblings, 1 reply; 4+ messages in thread
From: Nikhil Mohite @ 2020-10-08 09:34 UTC (permalink / raw)
To: Dave Page <[email protected]>; +Cc: pgadmin-hackers
Hi Dave,
On Thu, Oct 8, 2020 at 2:11 PM Dave Page <[email protected]> wrote:
> Hi
>
> On Thu, Oct 8, 2020 at 6:48 AM Nikhil Mohite <
> [email protected]> wrote:
>
>> Hi Team,
>>
>> Regarding feature RM-4232 <https://redmine.postgresql.org/issues/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: 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: 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?
>
We can provide separate placeholders for tooltips, but currently, we
are just adding the "Query Tool - " or "View/Edit Data - " prefix in the
tooltip.
e.g: ("Query Tool - TestDB/postgres@postgres-12") the remaining string
is similar to the title string.
>
>>
>> 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
>
>
Regards,
Nikhil Mohite.
Attachments:
[image/png] image.png (288.7K, 3-image.png)
download | view image
[image/png] image.png (157.6K, 4-image.png)
download | view image
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion.
2020-10-08 05:47 RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Nikhil Mohite <[email protected]>
2020-10-08 08:41 ` Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Dave Page <[email protected]>
2020-10-08 09:34 ` Re: RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Nikhil Mohite <[email protected]>
@ 2020-10-08 09:39 ` Dave Page <[email protected]>
0 siblings, 0 replies; 4+ messages in thread
From: Dave Page @ 2020-10-08 09:39 UTC (permalink / raw)
To: Nikhil Mohite <[email protected]>; +Cc: pgadmin-hackers
On Thu, Oct 8, 2020 at 10:34 AM Nikhil Mohite <
[email protected]> wrote:
> Hi Dave,
>
> On Thu, Oct 8, 2020 at 2:11 PM Dave Page <[email protected]> wrote:
>
>> Hi
>>
>> On Thu, Oct 8, 2020 at 6:48 AM Nikhil Mohite <
>> [email protected]> wrote:
>>
>>> Hi Team,
>>>
>>> Regarding feature RM-4232 <https://redmine.postgresql.org/issues/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: 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: 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?
>>
>
> We can provide separate placeholders for tooltips, but currently, we
> are just adding the "Query Tool - " or "View/Edit Data - " prefix in the
> tooltip.
> e.g: ("Query Tool - TestDB/postgres@postgres-12") the remaining
> string is similar to the title string.
>
OK, let's leave that as-is then.
Thanks!
>
>
>>
>>>
>>> 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
>>
>>
> Regards,
> Nikhil Mohite.
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EDB: http://www.enterprisedb.com
Attachments:
[image/png] image.png (288.7K, 3-image.png)
download | view image
[image/png] image.png (157.6K, 4-image.png)
download | view image
^ permalink raw reply [nested|flat] 4+ messages in thread
end of thread, other threads:[~2020-10-08 09:39 UTC | newest]
Thread overview: 4+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2020-10-08 05:47 RM-4232- Change what is shown by default in tab titles: Implementation suggestion. Nikhil Mohite <[email protected]>
2020-10-08 08:41 ` Dave Page <[email protected]>
2020-10-08 09:34 ` Nikhil Mohite <[email protected]>
2020-10-08 09:39 ` Dave Page <[email protected]>
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox