Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"mark as read" has a ~1sec delay #2332

Closed
sleeksorrow opened this issue Apr 3, 2019 · 9 comments
Closed

"mark as read" has a ~1sec delay #2332

sleeksorrow opened this issue Apr 3, 2019 · 9 comments
Milestone

Comments

@sleeksorrow
Copy link

sleeksorrow commented Apr 3, 2019

After updating to 1.14.0 marking articles as read has a delay of about 1 second until the "eye" icon and the headline turns grey. The other way around, marking as unread, has immediate visual feedback.

Maybe marking as read is not handled asynchronously anymore? Maybe intended?

@Alkarex
Copy link
Member

Alkarex commented Apr 3, 2019

Yes, intended, and even more asynchronous than before

#2310 (comment)

Regarding the delay, it is because marking as read is now sent in batch, max once per second. You can mark many as read or even scroll down without waiting.

We will follow-up in #2304 instant visual feedback to users

@Alkarex Alkarex closed this as completed Apr 3, 2019
@Alkarex Alkarex added this to the 1.15.0 milestone Apr 3, 2019
@oupala
Copy link
Contributor

oupala commented Apr 3, 2019

I'm invitating myself into this issue, sorry if it's not right solution.

I've just updated to version 1.14.0 and it appears that an article does no longer disappear when marked as read. I have to refresh the page (with F5) to see the that the article has disappeared from the unread article list.

Is this the expected behavior?

@Frenzie
Copy link
Member

Frenzie commented Apr 3, 2019

@oupala This is with the Hide article after reading setting checked? That's in Settings → Reading.

@Alkarex
Copy link
Member

Alkarex commented Apr 3, 2019

@oupala Yes, there is a bug to fix #2323

@sleeksorrow
Copy link
Author

Thank you for the explanation. I can live with this behavior until #2304 is in place. It's just the impression of unresponsiveness, but after all it's all usable as before.

@oupala
Copy link
Contributor

oupala commented Apr 3, 2019

@Frenzie Yes, Hide article after reading is checked. I'll check issue #2323.

Sorry for squatting this issue.

Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Apr 6, 2019
FreshRSS#2332
FreshRSS#2345
Re-introduces the instant-remove article.
Batch mark-as-read only used for fast actions like scroll and keyboard
shortcut for next/previous articles.
@Alkarex
Copy link
Member

Alkarex commented Apr 6, 2019

Could you please try #2349 ?

@sleeksorrow
Copy link
Author

Applied to dev branch and works like a charm for me. Thank you!

@Alkarex
Copy link
Member

Alkarex commented Apr 6, 2019

Thanks for the quick feedback :-)

Alkarex added a commit that referenced this issue Apr 6, 2019
#2332
#2345
Re-introduces the instant-remove article.
Batch mark-as-read only used for fast actions like scroll and keyboard
shortcut for next/previous articles.
@Alkarex Alkarex modified the milestones: 1.15.0, 1.14.1 Apr 6, 2019
Alkarex added a commit that referenced this issue Apr 6, 2019
javerous pushed a commit to javerous/FreshRSS that referenced this issue Jan 20, 2020
FreshRSS#2332
FreshRSS#2345
Re-introduces the instant-remove article.
Batch mark-as-read only used for fast actions like scroll and keyboard
shortcut for next/previous articles.
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this issue Mar 25, 2021
FreshRSS#2332
FreshRSS#2345
Re-introduces the instant-remove article.
Batch mark-as-read only used for fast actions like scroll and keyboard
shortcut for next/previous articles.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants