Reader View for Pale Moon

Add-ons for Pale Moon and other applications
General discussion, compatibility, contributed extensions, themes, plugins, and more.

Moderators: FranklinDM, Lootyhoof

User avatar
jars_
Lunatic
Lunatic
Posts: 397
Joined: 2016-12-27, 00:12

Re: Reader View for Pale Moon

Unread post by jars_ » 2017-12-08, 06:34

Huh, I just installed this add-on to try it out, and all but one of the add-on buttons in my navigation toolbar disappeared
same thing with me :D oh-oh-oh...

User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-08, 07:11

Version 1.0.1 removed from the site until this gets resolved. Sorry for the trouble, folks! Please revert to 1.0.0 for the time being.
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

User avatar
gracious1
Keeps coming back
Keeps coming back
Posts: 891
Joined: 2016-05-15, 05:00
Location: humid upstate NY

Re: Reader View for Pale Moon

Unread post by gracious1 » 2017-12-08, 11:23

I did some further testing in a clean profile (called "test04").
  1. Create custom toolbar Foo Bar.
  2. Place several buttons from Customize dialog box into Foo Bar.
  3. Restart Pale Moon.
  4. Foo Bar with buttons is still there.
  5. Install Reader View 1.0.1
  6. Restart Pale Moon.
  7. Foo Bar is gone; buttons are back in Customize dialog box. :thumbdown:

    Now here is the interesting bit…
  8. Open Add-ons Manager and hit Reader View's "Remove" button.
  9. Restart Pale Moon.
  10. Voila! Foo Bar is back with all its buttons! :thumbup:
This did not work in the working profile (called SEPTIMUS) that I had been using however, because I had used Clean Uninstall to remove preferences, as stated in my previous post. So maybe that was overkill and further damage was done. Also re-installing 1.0.0 seemed not work either. (I have not tested that yet in profile "test04".) And now, SEPTIMUS can't keep any custom toolbars after restart, so something is quite wrong. :problem:

UPDATE: I removed RV 1.0.0 from SEPTIMUS, and now it retains custom toolbars after restart. So it look likes even version 1.0.0 of RV is the culprit. Whoops. So it's doing the same thing that 1.0.1 does.
UPDATE_2: Verified that same behavior as occurred in test above occurs also in with Reader View 1.0.0 (disappearing custom toolbar).

Ubuntu 14.04 | PM 27.7.0a2 (64-bit) (2017-11-23)
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀

User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-10, 05:04

I have done some testing with various add-ons as well as my own on both my Windows 7 machine and my Linux partition with regards to the aforementioned issues, and have had pretty mixed results. Please see my comments on the repository here and here. (Feel free to reply on this thread, I just don't want to post the same information twice in two places.)

In summary, I think some of these issues are simply beyond my control. For other toolbar buttons moving or being removed, that should at least be a one-time deal. There's now a first_run preference in about:config for this extension that is set to "false" automatically the first time you install the extension. So, even if you remove it and reinstall it, it won't try to add or move its button anywhere unless you reset the preference. If this still concerns you, I've outlined a step you can follow in that comment.

For custom toolbars being hidden... I wish I had an answer. I'll keep playing with my code to see if I can determine what's causing it, but I don't have a good response for that issue just yet. Thank you for your efforts, gracious1. As I mentioned, I was able to trigger this behavior with other add-ons besides Reader View, and what add-ons trigger it seem to be somewhat random.

I'll probably be releasing v1.0.2 tomorrow. It will contain most of the fixes and requests that were in v1.0.1, and should fix at least some of the issues around toolbar buttons. Thanks for the feedback, everyone!
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

User avatar
SpockFan02
Astronaut
Astronaut
Posts: 535
Joined: 2017-09-24, 16:35
Location: Mak pupulusšum, California

Re: Reader View for Pale Moon

Unread post by SpockFan02 » 2017-12-10, 08:55

gracious1 wrote:...custom toolbar Foo Bar. ...
Hehehe...

User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-10, 15:24

Reader View 1.0.2 is now up. The Pale Moon addons page and the github repository have both been updated. Please see the release notes for details.

This should fix some of the issues with add-on buttons being moved on first install (where the first_run preference is set to true or does not exist) as well as with custom toolbars disappearing.

Thank you everyone for the feedback, and happy reading!
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

User avatar
[DarKwiN]
Newbie
Newbie
Posts: 5
Joined: 2017-08-30, 08:34

Re: Reader View for Pale Moon

Unread post by [DarKwiN] » 2017-12-14, 16:21

I found another problem related to toolbar: when installing Reader View, the refresh/stop button I've got on the right end of URL box gets thrown out of the URL box and divided in two split buttons (the refresh one and the stop one).
Disabling Reader View fixes the problem. Attached to the post you'll find a screenshot describing the situation.
Thanx!!
Attachments
refresh.png
refresh.png (14.48 KiB) Viewed 2712 times

User avatar
gracious1
Keeps coming back
Keeps coming back
Posts: 891
Joined: 2016-05-15, 05:00
Location: humid upstate NY

Re: Reader View for Pale Moon

Unread post by gracious1 » 2017-12-15, 00:28

[DarKwiN] wrote:I found another problem related to toolbar: when installing Reader View, the refresh/stop button I've got on the right end of URL box gets thrown out of the URL box and divided in two split buttons (the refresh one and the stop one).
Disabling Reader View fixes the problem. Attached to the post you'll find a screenshot describing the situation.
Thanx!!
So your toolbar looks like this…
Image

Now try this:
  1. Right-click in the toolbar and select "Customize…"
    Image
  2. While the Customize dialog is open, move the Reader button to the right of Reload button.
    Image
    In other words, the order of the buttons is now Reader View, Reload, and Stop.
  3. Click the "Done" button in the Customize dialog.
  4. Now observe that the Stop and Reload buttons are no longer separate but unified again.
    Image
Whenever the Stop button is placed to the right of the Reload, they will unify. When the Stop button is to the left of Reload, or when a button is placed between them, they will remain separate.
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀

User avatar
[DarKwiN]
Newbie
Newbie
Posts: 5
Joined: 2017-08-30, 08:34

Re: Reader View for Pale Moon

Unread post by [DarKwiN] » 2017-12-15, 09:12

Nice tip, thank you!
I further examined the issue, and I could pin down the problem to simply not putting Reader View (or any other) button between Reload and Stop. Anything between them not only cause them to split, but also prevent the Reload button to be nicely put in the URL box. No problem putting anything to the left of the Reload button.
Thanx again!

User avatar
gracious1
Keeps coming back
Keeps coming back
Posts: 891
Joined: 2016-05-15, 05:00
Location: humid upstate NY

Re: Reader View for Pale Moon

Unread post by gracious1 » 2017-12-15, 09:43

[DarKwiN] wrote:Nice tip, thank you!
I further examined the issue, and I could pin down the problem to simply not putting Reader View (or any other) button between Reload and Stop.
I think I said that in the post above.... :shock: ;)
[DarKwiN] wrote:Anything between them not only cause them to split, but also prevent the Reload button to be nicely put in the URL box. No problem putting anything to the left of the Reload button.
Anything but the stop button (if you want them to be unified)!
Thanx again!
You're welcome.
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀

User avatar
fatboy
Astronaut
Astronaut
Posts: 556
Joined: 2017-12-19, 08:03
Location: Canada

Re: Reader View for Pale Moon

Unread post by fatboy » 2017-12-19, 11:01

This is probably my most favourite add-on and feature. However, I am observing "problems" with the add-on: I have to frequently reload pages for a page to become readable by the add-on.
Systemd Free - MX Linux, Antix Linux & Artix Linux

User avatar
jars_
Lunatic
Lunatic
Posts: 397
Joined: 2016-12-27, 00:12

Re: Reader View for Pale Moon

Unread post by jars_ » 2017-12-19, 11:33

have to frequently reload pages for a page to become readable by the add-on.]
yes, the extension does not work for pages loaded in background, and because of this I made a separate button:

Code: Select all

ReaderParent.toggleReaderMode(event);
Url-button with icon

Code: Select all

custombutton://%3C%3Fxml%20version%3D%221.0%22%20encoding%3D%22UTF-8%22%3F%3E%0D%0A%3Ccustombutton%20xmlns%3Acb%3D%22http%3A//xsms.nm.ru/custombuttons/%22%3E%0A%20%20%3Cname%3EReader%20View%20%28%u043E%u0442%u0434%u0435%u043B%u044C%u043D%u0430%u044F%20%u043A%u043D%u043E%u043F%u043A%u0430%29%3C/name%3E%0A%20%20%3Cimage%3E%3C%21%5BCDATA%5Bdata%3Aimage/svg+xml%3Bbase64%2CPD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4NCjwhLS0gVGhpcyBTb3VyY2UgQ29kZSBGb3JtIGlzIHN1YmplY3QgdG8gdGhlIHRlcm1zIG9mIHRoZSBNb3ppbGxhIFB1YmxpYw0KICAgLSBMaWNlbnNlLCB2LiAyLjAuIElmIGEgY29weSBvZiB0aGUgTVBMIHdhcyBub3QgZGlzdHJpYnV0ZWQgd2l0aCB0aGlzDQogICAtIGZpbGUsIFlvdSBjYW4gb2J0YWluIG9uZSBhdCBodHRwOi8vbW96aWxsYS5vcmcvTVBMLzIuMC8uIC0tPg0KPHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiB3aWR0aD0iMTgiIGhlaWdodD0iMTgiIHZpZXdCb3g9IjAgMCAxNiAxNiI+DQogIDxkZWZzPg0KICAgIDxwYXRoIGlkPSJnbHlwaFNoYXBlLXJlYWRlck1vZGUtYm9vayIgZD0iTTUuNSw1aC0yQzMuMiw1LDMsNS4yLDMsNS41UzMuMiw2LDMuNSw2aDIgQzUuOCw2LDYsNS44LDYsNS41UzUuOCw1LDUuNSw1eiBNNS41LDdoLTJDMy4yLDcsMyw3LjIsMyw3LjVTMy4yLDgsMy41LDhoMkM1LjgsOCw2LDcuOCw2LDcuNVM1LjgsNyw1LjUsN3ogTTUuNSw5aC0yIEMzLjIsOSwzLDkuMiwzLDkuNVMzLjIsMTAsMy41LDEwaDJDNS44LDEwLDYsOS44LDYsOS41UzUuOCw5LDUuNSw5eiBNMTUuNCwyYzAsMC0zLjEsMC00LjQsMFM4LjEsMi41LDgsNC4zQzcuOSwyLjUsNi4zLDIsNSwyIFMwLjYsMiwwLjYsMkMwLjMsMiwwLDIuMywwLDIuN3Y5LjZDMCwxMi42LDAuMywxMywwLjYsMTNjMCwwLDIuNiwwLDQuNCwwYzEuNiwwLDIuOCwxLDMsMi4zQzguMiwxNCw5LjQsMTMsMTEsMTMgYzEuOCwwLDQuNCwwLDQuNCwwYzAuNCwwLDAuNi0wLjQsMC42LTAuOFYyLjdDMTYsMi4zLDE1LjcsMiwxNS40LDJ6IE0xNCwxMUwxNCwxMWMtMC4yLDAtMS42LDAtMywwYy0xLjYsMC0yLjksMC44LTMsMi4yIEM3LjksMTEuOCw2LjYsMTEsNSwxMWMtMS40LDAtMi44LDAtMywwbDAsMGwwLDBWNGMwLDAsMi43LDAsMy41LDBDNi42LDQsOCw1LjUsOCw2LjhDOCw1LjUsOS40LDQsMTAuNSw0QzExLjMsNCwxNCw0LDE0LDRWMTEgTDE0LDExeiIvPg0KICAgIDxsaW5lYXJHcmFkaWVudCBpZD0iZ3JhZGllbnQtc3RhdGUtcHJlc3NlZCIgeDE9IjAlIiB5MT0iMCUiIHgyPSIwIiB5Mj0iMTAwJSI+DQogICAgICA8c3RvcCBzdG9wLWNvbG9yPSIjNmY2ZmZmIiBvZmZzZXQ9IjAlIi8+DQogICAgICA8c3RvcCBzdG9wLWNvbG9yPSIjMjQyNGZmIiBvZmZzZXQ9IjEwMCUiLz4NCiAgICA8L2xpbmVhckdyYWRpZW50Pg0KICAgIDxzdHlsZT4NCiAgICAgIC5pY29uLXN0YXRlLXByZXNzZWQgeyBmaWxsOiB1cmwoI2dyYWRpZW50LXN0YXRlLXByZXNzZWQpOyB9DQogICAgPC9zdHlsZT4NCiAgPC9kZWZzPg0KICA8dXNlIHhsaW5rOmhyZWY9IiNnbHlwaFNoYXBlLXJlYWRlck1vZGUtYm9vayIgY2xhc3M9Imljb24tc3RhdGUtcHJlc3NlZCIvPg0KPC9zdmc+DQo%3D%5D%5D%3E%3C/image%3E%0A%20%20%3Cmode%3E0%3C/mode%3E%0A%20%20%3Cinitcode%3E%3C%21%5BCDATA%5B/*Initialization%20Code*/%5D%5D%3E%3C/initcode%3E%0A%20%20%3Ccode%3E%3C%21%5BCDATA%5BReaderParent.toggleReaderMode%28event%29%3B%5D%5D%3E%3C/code%3E%0A%20%20%3Caccelkey%3E%3C%21%5BCDATA%5B%5D%5D%3E%3C/accelkey%3E%0A%20%20%3Chelp%3E%3C%21%5BCDATA%5B%5D%5D%3E%3C/help%3E%0A%20%20%3Cattributes/%3E%0A%3C/custombutton%3E

User avatar
gracious1
Keeps coming back
Keeps coming back
Posts: 891
Joined: 2016-05-15, 05:00
Location: humid upstate NY

Re: Reader View for Pale Moon

Unread post by gracious1 » 2017-12-19, 12:24

jars_ wrote: yes, the extension does not work for pages loaded in background, and because of this I made a separate button:

Code: Select all

ReaderParent.toggleReaderMode(event);
Where exactly do you put the code?
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀

User avatar
jars_
Lunatic
Lunatic
Posts: 397
Joined: 2016-12-27, 00:12

Re: Reader View for Pale Moon

Unread post by jars_ » 2017-12-19, 13:42


User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-19, 17:01

I am currently working on a fix for this. It has to do with some of the code I borrowed from the original browser's codebase working differently in the context of an extension. I have a tentative fix in place in my development environment, but I want to do some more testing to make sure it's ready.

For now, selecting a background-loaded tab and refreshing it should cause the button to update correctly. Force-toggling Reader Mode also works I guess, but it should not be necessary after the next update.

Edit: The issue is also on the Github repo, #15.
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

User avatar
fatboy
Astronaut
Astronaut
Posts: 556
Joined: 2017-12-19, 08:03
Location: Canada

Re: Reader View for Pale Moon

Unread post by fatboy » 2017-12-23, 15:43

Isengrim wrote:I am currently working on a fix for this. It has to do with some of the code I borrowed from the original browser's codebase working differently in the context of an extension. I have a tentative fix in place in my development environment, but I want to do some more testing to make sure it's ready.

For now, selecting a background-loaded tab and refreshing it should cause the button to update correctly. Force-toggling Reader Mode also works I guess, but it should not be necessary after the next update.

Edit: The issue is also on the Github repo, #15.
Wow, thank you for the quick reply and quick fix! :)
Systemd Free - MX Linux, Antix Linux & Artix Linux

User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-29, 17:58

Reader View 1.1.0 is now up. The Pale Moon addons page and the github repository have both been updated. Please see the release notes for details.

Most of the changes are under-the-hood updates from the upstream mozilla-central code. I've also added localization files pulled from l10n-central. Most of these will be incomplete, but they're still applicable enough that I went ahead and included them.

As a side note, I've also added xpi files to the github release pages so you can download them directly if you wish.

Feliz lectura! Joyeuse lecture! Buona lettura!
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

f-117
Moonbather
Moonbather
Posts: 70
Joined: 2017-02-04, 20:41

Re: Reader View for Pale Moon

Unread post by f-117 » 2017-12-29, 19:38

So, what will you be working on next in the github "Issues" list?

Scott.
Last edited by f-117 on 2017-12-29, 19:58, edited 1 time in total.

User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-29, 20:00

f-117 wrote:So, what will you be working on next in the "Issues" list?
Well, I just realized that all of the localizations are incomplete, which will cause some breakage... so I'm roughing in some missing translations first. :crazy:

The next highest priority is #15. It's going to take a bit of rework and plenty of testing, so that one may be a little while.

Beyond that, #7 if it's feasible, or else I'll probably look at #16.
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

User avatar
Isengrim
Board Warrior
Board Warrior
Posts: 1325
Joined: 2015-09-08, 22:54
Location: 127.0.0.1
Contact:

Re: Reader View for Pale Moon

Unread post by Isengrim » 2017-12-29, 21:12

Reader View 1.1.1 is now up. The Pale Moon addons page and the github repository have both been updated. Please see the release notes for details.

This is an update/fix for several of the previously included localizations. The locale files were missing translations for a string property I added to the fork, which would cause the add-on to not function correctly if you were using any locale other than en-US. I've added some very rough translations for this text to at least fix functionality, but they probably need some additional work - see #21.

Happy reading!
Last edited by Isengrim on 2017-12-29, 21:12, edited 1 time in total.
a.k.a. Ascrod
Linux Mint 19.3 Cinnamon (64-bit), Debian Bullseye (64-bit), Windows 7 (64-bit)
"As long as there is someone who will appreciate the work involved in the creation, the effort is time well spent." ~ Tetsuzou Kamadani, Cave Story

Locked