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 only works halfway on newest 3.9 release #1727

Open
mlcampbe opened this issue Jan 23, 2024 · 9 comments
Open

Mark as read only works halfway on newest 3.9 release #1727

mlcampbe opened this issue Jan 23, 2024 · 9 comments
Labels

Comments

@mlcampbe
Copy link

Describe the bug
Prior to version 3.9 I could use (on my Mac) the command-k shortcut to mark all articles read. This inly works partly now in 3.9. I can still see that the number of unread articles on the sidebar list changes to zero but the articles shown in the right-hand pane are still showing the title in a bold font like an unread article.

Additionally if I use a filter to change between "all" and "unread" articles the righthand pane still shows all of the same ones in bold.

To Reproduce
Have a list of unread articles and press cmd-k

Screenshots
Add screenshots to help explain your problem, if possible.

Please complete the following information:

  • Vienna version 3.9
  • OS version: 14.2.1

Additional information:
In which version of Vienna does the problem not occur, if applicable.
Just started this behavior with the new 3.9 release.

@briandfoy
Copy link

I experience the same problem.

I also have a Smart Folder for all articles where is read is no. I can control click to get the contextual menu that includes "Mark All Articles as Read". When I do that, the article list pane does not update. If I choose another folder then come back to my Smart Folder, it shows all the articles as read. This seems like the article list fails to update even though they are correctly marked as read.

@mlcampbe
Copy link
Author

I'll note that I have found that if you click the refresh button after using cmd-k then the right-hand pane will clear and there are no articles shown. In the past pressing cmd-k would mark all as read and then refresh the display as well.

@josh64x2
Copy link
Member

I haven't been able to reproduce the original issue with the font staying bold when you press ⌘k, however I was able to reproduce the issue with the smart folder with both the keyboard shortcut and the contextual menu.

@briandfoy
Copy link

Btw, this is a bug, but it's not really bothering me since I check Vienna once a day. I know that the articles will be marked correctly, and since my usual next step is to quit Vienna, I don't need to see the pane updated. I can be patient for a fix. Other people may have different needs though. I should have noted that initially and I've felt a bit guilty about not mentioning that I don't want anyone to lose sleep to fix this right away for me.

@barijaona
Copy link
Member

Could you please specify which filter is set when you observe that behavior ?

@briandfoy
Copy link

In my case, the filter pull down says "All Articles" when I'm in my Smart Folder for is_read is no. I don't mess around with filters, so everything is whatever the default it.

@mlcampbe
Copy link
Author

Could you please specify which filter is set when you observe that behavior ?

For me I normally use the "unread articles" on the left-hand side panel while the is set for "all articles".

As briandfoy mentioned "If I choose another folder then come back to my unread Folder, it shows all the articles as read. "

@makigumo
Copy link

makigumo commented Apr 5, 2024

I maybe wrong but it seems the following line should also check for || currentFolder.type == VNAFolderTypeSmart.

if (currentFolder != nil && (currentFolder.type == VNAFolderTypeGroup || ![folderArray containsObject:currentFolder])) {

This seems to fix the issue with smart folders not visually updating for me.

@darth-hp
Copy link

Btw, this is a bug, but it's not really bothering me since I check Vienna once a day. I know that the articles will be marked correctly, and since my usual next step is to quit Vienna, I don't need to see the pane updated. I can be patient for a fix. Other people may have different needs though. I should have noted that initially and I've felt a bit guilty about not mentioning that I don't want anyone to lose sleep to fix this right away for me.

I have the same issue on MacOS 14.4.1 with Version 3.9.0 :7ad9df044: (8175) - marking all read has no visible effect. Clicking on an article flags it visibly read. Refreshing or switching to another view and back updates it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants