Closed Bug 771610 Opened 9 years ago Closed 9 years ago
Reader Mode: Added items to reader list items do not enter about:reader
Currently, added items to the Reader List are simply treated as alternative bookmarks. When one adds a page to the reader list and then proceeds to visit the page from the Reader List folder (in the Awesome Screen), the page does not enter about:reader. ER: Reader List opens added item about with an about:reader URI AR: Reader List item is opened (basically a bookmark) I used any article from http://neowin.net to test. I opened the article, opened my menu and selected 'Reader List'. I then went to the folder in the Awesome Screen and selected the item.
Reading list items currently work in the all pages tab; we also need to check for them in the bookmarks tab.
Assignee: nobody → bnicholson
Attachment #640863 - Flags: review?(margaret.leibovic)
Comment on attachment 640863 [details] [diff] [review] Open Reader Mode for Reading List bookmarks I feel like it would be nicer to do the exact same check we're doing in AllPagesTab , but it looks like our bookmarks query doesn't give us back Combined.DISPLAY, so we couldn't do that without changing the query, and I don't think it's worth doing that. Maybe you should just add a comment about what isInReadingList() actually means. I was initially confused that it didn't take a parameter, but it rather refers to the state of the bookmarks UI.  http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/awesomebar/AllPagesTab.java#202
Attachment #640863 - Flags: review?(margaret.leibovic) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Nightly (07/12), Samsung Galaxy Nexus (Android 4.1.1)
A new test case was created under the Reading Mode&Reading List test suite in the BFTs run: https://moztrap.mozilla.org/manage/cases/_detail/6364/
Flags: in-moztrap?(fennec) → in-moztrap+
You need to log in before you can comment on or make changes to this bug.