Opened on 11/13/2017 at 10:20:28 PM
Closed on 11/14/2017 at 10:21:49 PM
Last modified on 11/17/2017 at 06:10:35 PM
#6032 closed defect (fixed)
Page not added to the Reading List
| Reported by: | diegocarloslima | Assignee: | diegocarloslima |
|---|---|---|---|
| Priority: | P2 | Milestone: | Adblock-Browser-for-Android-1.3.2 |
| Module: | Adblock-Browser-for-Android | Keywords: | |
| Cc: | mario, tiago | Blocked By: | |
| Blocking: | Platform: | Adblock Browser for Android | |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | yes |
| Review URL(s): | |||
Description (last modified by diegocarloslima)
Environment
One Plus 5
Android 7.1.1
Adblock Browser 1.3.1
On Firefox 53.0.2, there's a regression, which causes pages to not be saved in the Reading List smart folder under the BOOKMARKS tab
How to reproduce
- Navigate to a page, e.g. https://adblockplus.org/blog/
- Tap the Reader View icon in the address bar
- Tap the menu overflow icon and select the star to add it to the bookmarks
These steps are also described into more details here
Observed behaviour
The page is saved under the bookmarks tab, but not included in the Reading List smart folder
Expected behaviour
The page should be included in the Reading List smart folder. We might use the fix made by Mozilla to avoid future merge conflicts
Attachments (0)
Change History (7)
comment:1 Changed on 11/13/2017 at 10:21:29 PM by diegocarloslima
- Summary changed from Page not added to Reading List to Page not added to the Reading List
comment:2 Changed on 11/13/2017 at 10:24:39 PM by diegocarloslima
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 11/14/2017 at 10:20:49 PM by abpbot
comment:5 Changed on 11/14/2017 at 10:21:49 PM by diegocarloslima
- Resolution set to fixed
- Status changed from reviewing to closed
comment:6 Changed on 11/15/2017 at 03:27:07 PM by mario
- Milestone set to Adblock-Browser-for-Android-1.3.2
comment:7 Changed on 11/17/2017 at 06:10:35 PM by traynard
- Verified working set
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 6032 - Page not added to the Reading List