Reset cursor on page change or category change

[f01c33]

Hi, so this is mainly a UX decision, of course, but i think the cursor should reset to 0 on page change or category change, it just makes more sense having a nice refresh on page changes, feel free to close if it does not make much sense to you :)

[bensadeh]

Hey @f01c33 and thanks for the PR!

I think this makes sense for category change, but for the page change I like the behavior of having the cursor the same place. Would you be able to update the PR for this?

[f01c33]

Sure!

[f01c33]

Changes done @bensadeh, it wasn't working correctly inside the:

if m.categoryHasStories(nextCat){

So i changed it to be above, and now it works fine

[bensadeh]

Thanks for updating it! I will have a look at as soon as I have the time, but the changes should be straightforward!

[bensadeh]

Sorry this took a while to get back to!

I just tried out the branch and there seems to be an issue when we change category the first time. On main, we dim the view and hide the selected cursor when change category to signalize that we are fetching new stories.

In your branch, the current page immediately goes blank. In other words, when we fetch a new category, the stories are now hidden instead of dimmed while waiting for the stories to fetch.