Re: [pgAdmin4][Patch]: RM #1978 - Add an option to allow user todisable alertifyjs and acitree animations - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgAdmin4][Patch]: RM #1978 - Add an option to allow user todisable alertifyjs and acitree animations
Date
Msg-id CAFOhELfXLBLuGM5stCZ3PTZhEeaYs9xwOPCAkb26hO8sAfL94A@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: RM #1978 - Add an option to allow user todisable alertifyjs and acitree animations  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch]: RM #1978 - Add an option to allow user todisable alertifyjs and acitree animations
List pgadmin-hackers
Hi,

Please find the attached updated patch.

On Thu, Mar 29, 2018 at 6:54 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Mar 29, 2018 at 1:51 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Mar 26, 2018 at 6:07 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Mar 26, 2018 at 7:23 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #1978: Add an option to allow user to disable alertifyjs and acitree animations.

I think these really need to be per-user settings, not per-installation.. Whether or not animations are shown is really a matter of personal taste and circumstance.

Right, it should be per-user settings.  Please find the attached updated patch. 

I found some issues I'm afraid:

- The label "Enable dialogues/notifications animation?" should read "Enable dialogue/notification animation?"

Changed. 
- Disabling treeview animation only seems to affect the main browser treeview, and not others in the application (e.g. the one on the Preferences panel).

Fixed 
 - After disabling dialogue/notification animations, I cannot re-enable notification animations. If I flip the switch back on, dialogue animations immediately start working again, but notification animations don't even work following a reload.

Fixed. 
Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Thanks,
Khushboo
Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: Proposal for changes in official Docker image
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch]: RM #1978 - Add an option to allow user todisable alertifyjs and acitree animations