Baxter900 / background-widget

An Ubersicht widget which adds a pretty animated background.
GNU General Public License v3.0
17 stars 3 forks source link

Performance improvement #2

Open imjosecampos-zz opened 8 years ago

imjosecampos-zz commented 8 years ago

Nice widget, have you thought about using a canvas? Right now this widget takes a lot of the processor not because it is doing some hard work but because the browser sucks at doing this without canvas, or even use TweenLite: https://greensock.com/tweenlite

Baxter900 commented 8 years ago

Hey, thanks! Sorry for the slow response, gmail put the notification into the "forums" tab, which I never look because nothing ends up there, so I didn't see this until today.

I honestly hadn't, but that's mostly because I'm rather horrible at web-scripting ^_^;

I haven't been using Ubersicht recently either, because it was conflicting with another application I was running.

I may try to bang my head against the wall and learn how to use Canvases later. I'm pretty bad at it though, so if you want to submit a pull that improves it, that's more than welcome!

Baxter900 commented 8 years ago

Alright, just ran a test, and as far as I can tell, due to the way Ubersicht is designed, I can't make the canvas anywhere near as efficient as the css animations. I can't get Ubersicht to only have to load the images once then re-use them when using a canvas (which should be possible, but I can't figure out how...) As you can guess, this hits the performance hard.

Anyway, if you can find a way to make it more efficient, just submit a pull request!