Reading list items not marked as read after viewing

RESOLVED FIXED

Status

()

Firefox for iOS
Reading List
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: callahad, Assigned: st3fan)

Tracking

unspecified
All
iOS

Firefox Tracking Flags

(fxios+)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
I don't seem to be able to mark my reading list items as read, no matter how many times I click on them, visit the link, or click the little blue circle in the reader view.

iOS 8.2 / Build 5 / 5th Generation iPod Touch

Updated

3 years ago
tracking-fennec: --- → ?
(Assignee)

Updated

3 years ago
Assignee: nobody → sarentz
Blocks: 1130493
(Assignee)

Comment 1

3 years ago
It is not implemented yet. But lets keep this on file.
tracking-fennec: ? → +
(Assignee)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
This is still problematic. Items one adds to one's reading list will never be marked as unread.
Status: RESOLVED → REOPENED
tracking-fennec: --- → ---
tracking-fxios: --- → ?
Resolution: FIXED → ---
You can mark items as read manually but they are never automatically marked as read. I'm not sure if this is design intentional but this goes against user expectation.
Summary: Items not marked (or markable) as read → Items not marked as read
tracking-fxios: ? → +
(Assignee)

Updated

3 years ago
Status: REOPENED → ASSIGNED
OS: Mac OS X → iOS
Hardware: x86 → All
Summary: Items not marked as read → Reading list items not marked as read after viewing
(Assignee)

Comment 4

3 years ago
Created attachment 8633616 [details] [review]
PR: https://github.com/mozilla/firefox-ios/pull/734

This patch marks reading list items as read when you open them. This is a very basic start. Let me know via *UI Review* if this is good enough via v1.
Attachment #8633616 - Flags: ui-review?(randersen)
Attachment #8633616 - Flags: ui-review?(dhenein)
Attachment #8633616 - Flags: review?(etoop)
Attachment #8633616 - Flags: ui-review?(randersen) → ui-review+
Comment on attachment 8633616 [details] [review]
PR: https://github.com/mozilla/firefox-ios/pull/734

I would really like to see a test for this too.
Attachment #8633616 - Flags: review?(etoop) → review+
Attachment #8633616 - Flags: ui-review?(dhenein)
(Assignee)

Comment 6

3 years ago
Added a UI test. Merged.

https://github.com/mozilla/firefox-ios/commit/5b0f65795ee1b2d27445a2651a5e0a5b4f0f8c0d
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.