search

ellie-commons / sequeler

SQL Client built in Vala
GNU General Public License v3.0
804 stars 66 forks source link

Explicitly start and stop gtk spinners #328

Closed ender672 closed 4 years ago

ender672 commented 4 years ago

This cuts down on CPU usage when the application is idle.

Even when doing nothing, sequeler uses 1% to 2% CPU on my machine. On laptops this will prevent the CPU from going into idle states.

This happens when any part of the sequeler window is exposed.

I narrowed the CPU usage down to GTK spinners. It seems that the spinner animation is running even when the spinner is not visible.

This PR starts and stops spinners instead of letting them run all the time.

Alecaddd commented 4 years ago

This is interesting, I never noticed that. Thank you so much, I'll review it and merge it later. Cheers