elementary / calendar

Desktop calendar app designed for elementary OS
https://elementary.io
GNU General Public License v3.0
130 stars 39 forks source link

Use Libhandy for calandar pages #623

Open hanaral opened 3 years ago

hanaral commented 3 years ago

The current solution is slow and jarring even on my desktop, a lot of the time I don't even realise the month has changed when I scroll.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/95452403-use-libhandy-for-calandar-pages?utm_campaign=plugin&utm_content=tracker%2F60388692&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F60388692&utm_medium=issues&utm_source=github).
mcclurgm commented 3 years ago

To be clear, what exactly do you consider jarring? Is the animation stuttering or not playing for you? (If that's it, then it's unrelated to libhandy. It's a bigger performance issue.) Or are you seeing the proper animation where the months slide up or down, and you think that's jarring?

hanaral commented 3 years ago

To be clear, what exactly do you consider jarring? Is the animation stuttering or not playing for you? (If that's it, then it's unrelated to libhandy. It's a bigger performance issue.) Or are you seeing the proper animation where the months slide up or down, and you think that's jarring?

The animation does not play for me, due to the large amount of events there are on my calendar. I've tested this both on my machine which are running Mesa, so I don't think its a hardware thing. It seems that the view takes so long to load the next screen, there is a good few seconds between the month widget in the headerbar updating and the view moving

mcclurgm commented 3 years ago

Alright. I agree completely, that is jarring. I have tons of events on my calendar too, and it always takes a while to change pages.

This isn’t a libhandy issue though. (If you disable most of your calendars and switch pages, it’ll go smoothly.) So I’m going to close this because the real issue here is the performance of populating a month view, not the widget we use to do the swap.

danirabbit commented 3 years ago

I'd like to reopen this since I think we'd still get a better multitouch experience here using Hdy.Carousel