Re: Regarding RM #3316 Pgadmin4 No Tray Crash

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Regarding RM #3316 Pgadmin4 No Tray Crash
Date: 2018-07-11 07:03:38
Message-ID: CANxoLDeAVuRFFAJV=GnuoZ8q31zU5StZJLT0ucEprB7Od68UbQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Tue, Jul 10, 2018 at 9:16 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Tue, Jul 10, 2018 at 4:05 PM, Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>>
>>
>> On Tue, 10 Jul 2018, 18:32 Dave Page, <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Tue, Jul 10, 2018 at 11:20 AM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> I have implemented the fix for RM #3316 "Pgadmin4 No Tray Crash". I
>>>> have implemented it as follows:
>>>>
>>>> - Check the availability of System Tray for 30 seconds (no changes
>>>> made here).
>>>> - If System Tray not found create one Floating Window with menu.
>>>> (Refer attached screenshot).
>>>> - I have remove close(x) button of the floating window.
>>>> - Fedora have "Quit" menu for the applications, so I have handle
>>>> the close event and shutdown the python server.
>>>>
>>>> I have tested this on Fedora-28 (no system tray) and Ubuntu 18.04
>>>> (with system tray).
>>>>
>>>> Please review the screenshot and suggest changes (if any). I'll send
>>>> the patch later.
>>>>
>>>
>>> What does the window look like without the menu? I'd suggest putting a
>>> Slonik image there, and maybe a note (or a button to display a note) saying
>>> that installing a system tray plugin can be used to hide the window.
>>>
>>
>> You mean instead of "pgAdmin4" menu we should add toolbar with
>> button having slonik image and when click on there submenus will be open.
>>
>
> No - I assume that's a drop down menu over a blank window? I'm suggesting
> something to fill the blank space.
>

Attached are the screenshot after adding 'Slonik' icon and 'Note'.
Should we allow user to resize the floating window? Definitely not
maximize, because icon gets blurred.

>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

--
*Akshay Joshi*

*Sr. Software Architect *

*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*

Attachment Content-Type Size
image/png 70.1 KB
pgAdmin4_WithMenu_Open.png image/png 109.1 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-07-11 08:01:18 Re: Regarding RM #3316 Pgadmin4 No Tray Crash
Previous Message Khushboo Vashi 2018-07-11 07:03:13 Re: [pgAdmin4][Patch]: Fix RM #3191 : Debug option is not working