Hi On Thu, Mar 29, 2018 at 1:51 PM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote:
> > > On Mon, Mar 26, 2018 at 6:07 PM, Dave Page <dp...@pgadmin.org> wrote: > >> Hi >> >> On Mon, Mar 26, 2018 at 7:23 AM, Khushboo Vashi < >> khushboo.va...@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?" - 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). - 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. Thanks. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
