Posts: 1,316
Joined: Mar 2018
Reputation:
29
Thanks for testing. I don‘t think the minimal stutter you describe is new, but simply caused by the fact that rounded edges needs some more processing time. I can look into further optimization, but first I wanted to get back to the status before sdwebimage 3.8.3. Is this reached from your experience (related to both stutter and resolution)?
Posts: 495
Joined: Apr 2014
Reputation:
8
Yes I just finished testing the iPad. It works great there. And the slow loading I complained about with movie details (related to my high resolution images) doesn’t seem to be a problem anymore either! So resolution issue is resolved entirely for me!
So I was testing the stutter issue again today and I discovered a plot twist.
Stutter only happens when in power saving mode. While it isn’t surprising that the cpu would be more strained in power save mode, no stutter is present in the full movie list where tons of items are visible.
Yet in the season/episode view for shows, even when there is no artwork, with rounded corners there is stutter. I don’t understand why a view with no, or only a couple of images would stutter when one with many does not.
Posts: 495
Joined: Apr 2014
Reputation:
8
That makes sense. I was thinking the images were being cached but since they’re not and having to re-round the images each time is certainly not an ideal solution. Todays devices can certainly handle it but since you are supporting very old devices still I wonder how those perform.
Great to see these kinds things being worked on and 3rd party libraries successfully updated. While invisible to the end user, it’s satisfying knowing that the app is being made more robust and future proof with these changes.
If you really want to challenge yourself, may I suggest you revisit the collapsing and expanding season/episode view? It really does some wonky things. Especially if you expand and collapse out of order, it just feels so janky. I personally feel it would benefit from a refined experience. 🙂
Posts: 495
Joined: Apr 2014
Reputation:
8
I have a few ideas.
When you expand multiple seasons I really think the header for the most recently expanded season should be pushed to the top of the view. Currently it only pushed up a few notches displaying two episodes. One would think of you are expanding a season you are interested in viewing as much of that season right from the start. I also believe this would avoid that distracting expand from the top animation.
Secondly, if you have multiple seasons expanded and you collapse from the bottom it always brings you back to season one. I think it would be a lot nicer if it brought you to the next preceding season that is expanded, sliding that seasons header to the top of the view as described in the first suggestion.
Even if you expand a season when the preceding seasons are collapsed and have a season below it expanded, the season header you expand will slide down revealing less episodes than if it simply stayed put.
Posts: 1,316
Joined: Mar 2018
Reputation:
29
Thanks, need to think about this and play around a bit. I am not a regular user of this menu. The current implementation tries to scroll the unfolded episodes into view. All other scrolling is handled by iOS. Of course this can be change to e.g. only having one season expanded (collapsing others when expanding a new one) and then move the section header to top. It is just about what makes best sense from the use case.
Btw, this has nothing to do with this "scrolling in from top" issue, which is just an animation setting.
Posts: 495
Joined: Apr 2014
Reputation:
8
Only having one season able to be expanded is another good idea I can get behind.
I’d still like it to slide the season header to the top upon expansion to make as many episodes visible as possible.
Posts: 495
Joined: Apr 2014
Reputation:
8
2023-06-18, 22:54
(This post was last modified: 2023-06-18, 22:57 by amasephy. Edited 1 time in total.)
Is there a possible third hybrid option? Here’s what I like. The “with animation” has a nice unfolding appearance and appears less jumpy. But, I don’t like the briefly visible black background. it’s more visible when collapsing the seasons. I find that distracting. The black background is not visible in the “without animation” version. If you can make it unfold nicely and not have the black background visible I think that would be the best presentation.
Edit: I had to look before I could confirm but I think my complaint really only applies to light mode. I think with dark mode the dark background would be expected as it matches the episode list items background.
Posts: 1,316
Joined: Mar 2018
Reputation:
29
2023-06-18, 23:17
(This post was last modified: 2023-06-19, 07:11 by Buschel. Edited 1 time in total.)
True, you see the app‘s background shortly. This background always has the same dark grey, which makes it less visible when using the app in dark mode. I will try, if other animation styles make this possibly less recognizable.
Btw, I also prefer the animated version. It is giving a smoother UI experience.
Edit: I played around with different animation methods and styles, but the temporary visibility of the background (this definitely more prominent when collapsing a season) is same. The only way to avoid this is to use a different way which does not animate at all. But this is not desired from two points of view: first, the animated version really feels smoother to use. Second, the non-animated approach requires the whole screen content to be recreated fresh (including reading and scaling thumbs ...) instead of only fasting in/out the season's episode.