Tunous / SwipeNews-Issues

Feedback and bug reporting for SwipeNews
7 stars 0 forks source link

Articles aren't marked as read, when tablet layout is enabled #92

Closed kontrastylez closed 7 years ago

kontrastylez commented 7 years ago

Version 0.20.2

Tunous commented 7 years ago

Pressing on mark as read button doesn't do anything?

kontrastylez commented 7 years ago

Manually marking as read works. But if you select an article in article list, so that it is opened in the right article content screen, the article isn't marked as read.

Tunous commented 7 years ago

That was actually done on purpose as I wasn't sure about the new behavior.

With this new split layout every time you open the app it automatically selects first article and displays it to the right. With auto marking as read on article click that article would be marked as read. So basically whenever you opened the app it always marked first article as read. I considered it potentially dangerous because if you accidently opened the app and didn't unmark the article it could disappear unnoticed.

Now I think that maybe instead of disabling this entirely I could only disable it for that single automatically opened article. Do you think that this would be a valid solution? Or maybe I'm just wrong and it's not necessary to disable this behavior at all...

kontrastylez commented 7 years ago

Disabling it only for the first article could be confusing for the user. I think the better solution would be, if the first article wouldn't be opened automatically.

Tunous commented 7 years ago

Improve article opening/marking as read with split layout

Tunous commented 7 years ago