On Tue, Apr 3, 2018 at 7:23 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi
Thanks - I've committed this, however, could you send me an updated screenshot for the docs? The one you sent was a different size and colour depth from the others (and looked like a different scale).
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.
