same thing with me oh-oh-oh...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
Reader View for Pale Moon
Moderators: FranklinDM, Lootyhoof
-
- Lunatic
- Posts: 399
- Joined: 2016-12-27, 00:12
Re: Reader View for Pale Moon
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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
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
-
- Keeps coming back
- Posts: 891
- Joined: 2016-05-15, 05:00
- Location: humid upstate NY
Re: Reader View for Pale Moon
I did some further testing in a clean profile (called "test04").
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)
- Create custom toolbar Foo Bar.
- Place several buttons from Customize dialog box into Foo Bar.
- Restart Pale Moon.
- Foo Bar with buttons is still there.
- Install Reader View 1.0.1
- Restart Pale Moon.
- Foo Bar is gone; buttons are back in Customize dialog box.
Now here is the interesting bit… - Open Add-ons Manager and hit Reader View's "Remove" button.
- Restart Pale Moon.
- Voila! Foo Bar is back with all its buttons!
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." 🚀
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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!
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
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
-
- Astronaut
- Posts: 535
- Joined: 2017-09-24, 16:35
- Location: Mak pupulusšum, California
Re: Reader View for Pale Moon
Hehehe...gracious1 wrote:...custom toolbar Foo Bar. ...
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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!
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
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
Re: Reader View for Pale Moon
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!!
Disabling Reader View fixes the problem. Attached to the post you'll find a screenshot describing the situation.
Thanx!!
You do not have the required permissions to view the files attached to this post.
-
- Keeps coming back
- Posts: 891
- Joined: 2016-05-15, 05:00
- Location: humid upstate NY
Re: Reader View for Pale Moon
So your toolbar looks like this…[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!!
Now try this:
- Right-click in the toolbar and select "Customize…"
- While the Customize dialog is open, move the Reader button to the right of Reload button.
In other words, the order of the buttons is now Reader View, Reload, and Stop. - Click the "Done" button in the Customize dialog.
- Now observe that the Stop and Reload buttons are no longer separate but unified again.
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀
Re: Reader View for Pale Moon
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!
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!
-
- Keeps coming back
- Posts: 891
- Joined: 2016-05-15, 05:00
- Location: humid upstate NY
Re: Reader View for Pale Moon
I think I said that in the post above....[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.
Anything but the stop button (if you want them to be unified)![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.
You're welcome.Thanx again!
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀
-
- Astronaut
- Posts: 558
- Joined: 2017-12-19, 08:03
- Location: Canada
Re: Reader View for Pale Moon
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
-
- Lunatic
- Posts: 399
- Joined: 2016-12-27, 00:12
Re: Reader View for Pale Moon
yes, the extension does not work for pages loaded in background, and because of this I made a separate button:have to frequently reload pages for a page to become readable by the add-on.]
Code: Select all
ReaderParent.toggleReaderMode(event);
-
- Keeps coming back
- Posts: 891
- Joined: 2016-05-15, 05:00
- Location: humid upstate NY
Re: Reader View for Pale Moon
Where exactly do you put the code?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);
20 July 1969 🌗 Apollo 11 🌓 "One small step for [a] man, one giant leap for mankind." 🚀
-
- Lunatic
- Posts: 399
- Joined: 2016-12-27, 00:12
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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.
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
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
-
- Astronaut
- Posts: 558
- Joined: 2017-12-19, 08:03
- Location: Canada
Re: Reader View for Pale Moon
Wow, thank you for the quick reply and quick fix!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.
Systemd Free - MX Linux, Antix Linux & Artix Linux
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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!
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
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
-
- Moonbather
- Posts: 72
- Joined: 2017-02-04, 20:41
Re: Reader View for Pale Moon
So, what will you be working on next in the github "Issues" list?
Scott.
Scott.
Last edited by f-117 on 2017-12-29, 19:58, edited 1 time in total.
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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.f-117 wrote:So, what will you be working on next in the "Issues" list?
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
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
-
- Board Warrior
- Posts: 1325
- Joined: 2015-09-08, 22:54
- Location: 127.0.0.1
Re: Reader View for Pale Moon
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!
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
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