Page action should toggle entries to reading list
kevonfernando opened this issue · 2 comments
The page action button should ideally be able to remove a webpage from the reading list if it has already been added.
Also this is a great addon. I really like the idea of keeping it in a native style which is why I downloaded it. Thank you for making this addon.
Hi! Thank you for a feedback!
I did this logic at the very beginning, but then I changed it - now clicking on the icon hoist a link in the list. I did it because it isn't possible to show in address bar the actual status (icon always in initial state) - so I had accidentally removed some of my articles.
I think about adding some indicator to show visited link in list. And maybe show some popup in opened link (but it requires access to all pages :( )
Oh, I see the problem. Maybe you could circumvent this issue by having two page actions? The current default (not added state) only shows when a url is not yet in the list and a new one (added state) only when it already is in the list. Would this be possible instead?
Also the read indicator in the list would be nice too. Would make it more like in Firefox for iOS.
And about the popup, the feature as well as the permission could be optional. This way the addon would be less scary for users.