Closed
Bug 1145388
Opened 10 years ago
Closed 9 years ago
Reading list items not marked as read after viewing
Categories
(Firefox for iOS :: Reading List, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fxios | + | --- |
People
(Reporter: callahad, Assigned: st3fan)
References
Details
Attachments
(1 file)
47 bytes,
text/x-github-pull-request
|
fluffyemily
:
review+
tecgirl
:
ui-review+
|
Details | Review |
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•10 years ago
|
tracking-fennec: --- → ?
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → sarentz
Blocks: iosreadinglist
Assignee | ||
Comment 1•10 years ago
|
||
It is not implemented yet. But lets keep this on file.
Updated•10 years ago
|
tracking-fennec: ? → +
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 2•9 years ago
|
||
This is still problematic. Items one adds to one's reading list will never be marked as unread.
Comment 3•9 years ago
|
||
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
Updated•9 years ago
|
Assignee | ||
Updated•9 years ago
|
Status: REOPENED → ASSIGNED
Updated•9 years ago
|
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•9 years ago
|
||
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)
Updated•9 years ago
|
Attachment #8633616 -
Flags: ui-review?(randersen) → ui-review+
Comment 5•9 years ago
|
||
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+
Updated•9 years ago
|
Attachment #8633616 -
Flags: ui-review?(dhenein)
Assignee | ||
Comment 6•9 years ago
|
||
Added a UI test. Merged.
https://github.com/mozilla/firefox-ios/commit/5b0f65795ee1b2d27445a2651a5e0a5b4f0f8c0d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•