Save Images by Lumox

General discussion, compatibility and contributed extensions.

Moderators: satrow, Lootyhoof, FranklinDM

TMINY
Apollo supporter
Apollo supporter
Posts: 43
Joined: 2016-02-02, 04:58

Save Images by Lumox

Unread post by TMINY » 2017-05-18, 15:44

I am on v1.1.03, but some of the functionality was lost... I think with the latest PM upgrade.

The Open Image Links in Tab works, but the Save Images does not save anything despite showing the results dialog (says zero saved).

Here is the latest version of the addon which won't install under PM-
https://addons.mozilla.org/en-US/firefo ... /versions/

I can install 1.2.6 under Moon Tester, but it doesn't save anything either and doesn't show the results dialog.

It passes the PM CompatChecker.

Any ideas on what it is dependent on?

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1772
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: Save Images by Lumox

Unread post by JustOff » 2017-05-19, 14:35

Since the original developer is going to abandon this add-on I've made the fork for Pale Moon: Save Images (Moon Edition).

Please note that I only checked the basic functionality, so if something doesn't work fill an issue on GitHub or report here.
Here are the add-ons I made in a spare time. That was fun!

TMINY
Apollo supporter
Apollo supporter
Posts: 43
Joined: 2016-02-02, 04:58

Re: Save Images by Lumox

Unread post by TMINY » 2017-05-19, 14:56

Wow! Thanks so much for forking this. It is working great for me!

There are two very minor bugs that were in the old plugin, but they are barely worth mentioning.

1. In in the context menu, if you select Save Images to the RIGHT or LEFT, it doesn't select that option in the Options dialog that opens. You have to choose it again manually.
2. If you select a folder in the destination picker, it duplicates the folder name in the path. You have to manually trim the end of the path.

Again, these are very minor.

Thanks again for helping me out with this!

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1772
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: Save Images by Lumox

Unread post by JustOff » 2017-05-19, 15:25

TMINY wrote:Wow! Thanks so much for forking this. It is working great for me!
Thanks for the confirmation.
There are two very minor bugs that were in the old plugin, but they are barely worth mentioning.
Ok, I'll take a look to it in a spare time.
Here are the add-ons I made in a spare time. That was fun!

Locked