Recensioni per Tranquility Reader
Tranquility Reader di arunk
Risposta di arunk
Replica dello sviluppatore
pubblicato il 4 anni faUpdate on 2020-09-24:
Links to related pages are restored in version 3.0.22 (released today).
The extension does try to scroll to the top of the opened page (even as of version 3.0.21); but the code did not seem to work for some cases; a change has been made to try and address this problem. If this problem persists, please provide a specific page with instructions on reproducing the problem.
-----------------------------
Update based on the example link provided:
Thanks for the example page. I identified that the regression happened between 3.0.18 (where this functionality worked for your page) and in 3.0.19 where it stopped working. I have added your example page to my regression tests suite.
The change that caused this regression does not have to do with the code that keeps track of such navigation related links; rather these links are enclosed within a certain type of tag that I started removing in 3.0.19 because for most pages, it was typically used as a way of navigating around the website as well as for social networking related links and had nothing to do with the main page content.
The fix for this regression will be available in the next release. It may take a couple of weeks for the next release so that I can combine this fix (moving just a couple of lines of code) with other fixes -- otherwise I will be burdening the reviewers with too many releases. Thanks for your patience.
-------------------------
Thanks for the review/feedback. 3.0.21 should not have any code that explicitly tries to remove the navigation links. Please update your comment with an example page or open a ticket on the github support page (https://github.com/ushnisha/tranquility-reader-webextensions/issues) and I can take a look.
The extension tries to save navigation links that match one of the following 18 patterns (comma separated pattern list):
NEXT,NEXT>,PREV,<PREV,PREVIOUS,<PREVIOUS,
NEXTPAGE,NEXTPAGE>,PREVIOUSPAGE,<PREVIOUSPAGE,
FIRST,<<FIRST,LAST,LAST>>,<,<<,>,>>
If the page has clickable navigation links that exactly match one of the above, the extension should be able to extract such links and preserve them in a clearly demarcated navigation section (with border); there is no guarantee that navigation links with other patterns will be retained by the extension -- although such links may have been retained in the past.
Links to related pages are restored in version 3.0.22 (released today).
The extension does try to scroll to the top of the opened page (even as of version 3.0.21); but the code did not seem to work for some cases; a change has been made to try and address this problem. If this problem persists, please provide a specific page with instructions on reproducing the problem.
-----------------------------
Update based on the example link provided:
Thanks for the example page. I identified that the regression happened between 3.0.18 (where this functionality worked for your page) and in 3.0.19 where it stopped working. I have added your example page to my regression tests suite.
The change that caused this regression does not have to do with the code that keeps track of such navigation related links; rather these links are enclosed within a certain type of tag that I started removing in 3.0.19 because for most pages, it was typically used as a way of navigating around the website as well as for social networking related links and had nothing to do with the main page content.
The fix for this regression will be available in the next release. It may take a couple of weeks for the next release so that I can combine this fix (moving just a couple of lines of code) with other fixes -- otherwise I will be burdening the reviewers with too many releases. Thanks for your patience.
-------------------------
Thanks for the review/feedback. 3.0.21 should not have any code that explicitly tries to remove the navigation links. Please update your comment with an example page or open a ticket on the github support page (https://github.com/ushnisha/tranquility-reader-webextensions/issues) and I can take a look.
The extension tries to save navigation links that match one of the following 18 patterns (comma separated pattern list):
NEXT,NEXT>,PREV,<PREV,PREVIOUS,<PREVIOUS,
NEXTPAGE,NEXTPAGE>,PREVIOUSPAGE,<PREVIOUSPAGE,
FIRST,<<FIRST,LAST,LAST>>,<,<<,>,>>
If the page has clickable navigation links that exactly match one of the above, the extension should be able to extract such links and preserve them in a clearly demarcated navigation section (with border); there is no guarantee that navigation links with other patterns will be retained by the extension -- although such links may have been retained in the past.