Re: Regarding RM #3316 Pgadmin4 No Tray Crash - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: Regarding RM #3316 Pgadmin4 No Tray Crash
Date
Msg-id CANxoLDeAVuRFFAJV=GnuoZ8q31zU5StZJLT0ucEprB7Od68UbQ@mail.gmail.com
Whole thread Raw
In response to Re: Regarding RM #3316 Pgadmin4 No Tray Crash  (Dave Page <dpage@pgadmin.org>)
Responses Re: Regarding RM #3316 Pgadmin4 No Tray Crash  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi

On Tue, Jul 10, 2018 at 9:16 PM, Dave Page <dpage@pgadmin.org> wrote:


On Tue, Jul 10, 2018 at 4:05 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:


On Tue, 10 Jul 2018, 18:32 Dave Page, <dpage@pgadmin.org> wrote:
Hi

On Tue, Jul 10, 2018 at 11:20 AM, Akshay Joshi <akshay.joshi@enterprisedb.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-9517
Mobile: +91 976-788-8246
Attachment

pgadmin-hackers by date:

Previous
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch]: Fix RM #3191 : Debug option is not working
Next
From: Dave Page
Date:
Subject: Re: Regarding RM #3316 Pgadmin4 No Tray Crash