universalviewer icon indicating copy to clipboard operation
universalviewer copied to clipboard

Makes disable animations work across all extensions' expand panels and makes it a global config

Open LlGC-jop opened this issue 1 year ago • 8 comments

Fixes #756

Reduced animations should now be available as a global config setting on the in-use extension so if any other modules have animations in future they can refer to it.

Also fixes the problem where the config didn't persist between refreshes.

Tested locally on OSD, PDF, Video, Audio, & 3D

LlGC-jop avatar Oct 22 '24 10:10 LlGC-jop

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
universalviewer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2024 0:51am

vercel[bot] avatar Oct 22 '24 10:10 vercel[bot]

Screen shot to clarify what I'm talking about: image

demiankatz avatar Oct 22 '24 11:10 demiankatz

In happier news, you can add "default extension" and "eBook extension" to the list of tested extensions where the setting does work as expected. :-)

demiankatz avatar Oct 22 '24 11:10 demiankatz

Thanks @demiankatz, not sure how I missed that. The code's right below the stuff I changed!

Should be ok now.

LlGC-jop avatar Oct 22 '24 12:10 LlGC-jop

There is one more I've found, the way the controls in OSD fade in/out, however I think that's a new issue as it's probably a setting in OSD (or needs one adding).

LlGC-jop avatar Oct 22 '24 13:10 LlGC-jop

Good point, @LlGC-jop! Do you want to open a new issue to track that one, and then once this is tested, we can merge it as incremental progress?

demiankatz avatar Oct 22 '24 13:10 demiankatz

@demiankatz @LlGC-jop Hi Jonathan and Demian - I tested the fix here and I can see a difference when the checkbox is ticked for change in motion for both OSD & PDF, however, I am less certain about the other item types - i.e. Video, Audio, & 3D as I struggle to see any difference in motion when opening and closing the side panels.

damodeburca avatar Oct 23 '24 10:10 damodeburca

@damodeburca, I just re-checked this video example. If the "reduce motion" box is checked in settings, the "more info" panel on the right snaps open and closed instantly. If the box is unchecked, you can see it sliding back and forth. Are you seeing different behavior? Is there a different manifest you are testing with that behaves differently?

demiankatz avatar Oct 23 '24 11:10 demiankatz

@demiankatz @LlGC-jop - Hi guys, I retested the 'reduce animation' check box with audio and 3D again and I am happy that this fix is working

damodeburca avatar Oct 28 '24 15:10 damodeburca

Thanks, @damodeburca and @LlGC-jop -- merging now!

demiankatz avatar Oct 28 '24 15:10 demiankatz