JSView 2.1.3 PM v27.1

Dedicated board for extension releases/support threads

Moderators: FranklinDM, Lootyhoof

Forum rules
Please do not create new topics here unless you are an extension author in need of a dedicated releases&support thread!
tmtisfree

JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-13, 00:39

Hi,

A working version of the good old JSView, a simple but powerful JS/CSS viewer for developers, revived for Pale Moon v27.1 (not tested on anything else).

Cheers,
TMTisFree
Attachments
jsview-2.1.3-pm_v27.xpi
JSView 2.1.3 PM v27.1
(122.53 KiB) Downloaded 59 times

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35473
Joined: 2011-08-28, 17:27
Location: Motala, SE
Contact:

Re: JSView 2.1.3 PM v27.1

Unread post by Moonchild » 2017-02-13, 00:48

(GPL v2.0 licensed.) If this works, you should get it included on the Pale Moon add-ons site and offer maintenance for it.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

New Tobin Paradigm

Re: JSView 2.1.3 PM v27.1

Unread post by New Tobin Paradigm » 2017-02-13, 02:52

Ensure the ID and branding has been changed to avoid technical and rights issues and follow: http://developer.palemoon.org/Add-ons:Site

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-13, 10:27

Matt A Tobin wrote:Ensure the ID and branding has been changed to avoid technical and rights issues and follow: http://developer.palemoon.org/Add-ons:Site
Thank you for the guidance.
Here is an updated version with your suggestion (new ID and name for inclusion in the Pale Moon add-ons repository:

Add-on Name: JSView Revive
64x64 alpha transparent PNG icon:
jsview-revive.png
jsview-revive.png (2.1 KiB) Viewed 3727 times
Author Name: Ron Beckman (Pale Moon - TMTisFree)
Slug Name: jsview-revive
Short Description: View the source code of external js and css files
Long Description: View the source code of external js and css files. Version adapted to work with Pale Moon v27 with cosmetic changes.
Initial XPI:
jsview-revive-2.1.4-pm_v27.xpi
(122.9 KiB) Downloaded 46 times
FTP User name: TMTisFree

Changelog for JSView post version 2.1.1 (TMTisFree)

JSView 2.1.2 (20151012):
+ Reverted back to GECKO < 27
+ Pale Moon support
+ Enhanced interface
JSView 2.1.3 (20170212):
+ Reverted back (once more) to GECKO > 27
+ Pale Moon 27 support
+ Enhanced Fr translation
JSView Revive 2.1.4 (20170213):
+ renamed as JSView Revive per Pale Moon support team request
+ changed ID for inclusion in Pale Moon add-on repository
+ cosmetic changes (some window titles not set; size columns in info window right-justified)
+ replaced the broken Mozilla add-on page URL with a link to the Pale Moon forum (will be changed to the Pale Moon repository link once added in)
+ Enhanced Fr translation

New Tobin Paradigm

Re: JSView 2.1.3 PM v27.1

Unread post by New Tobin Paradigm » 2017-02-13, 17:49

you need to send it to one of the add-ons team via private message not a post..

User avatar
Lootyhoof
Themeist
Themeist
Posts: 1568
Joined: 2012-02-09, 23:35
Location: United Kingdom

Re: JSView 2.1.3 PM v27.1

Unread post by Lootyhoof » 2017-02-13, 20:41

This should be fine for inclusion, however there is one small issue...

In install.rdf you have minVersion as 26.9. As this add-on is made for 27.1.0 and above, you should change the minVersion to reflect that.

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-14, 08:07

Lootyhoof wrote:This should be fine for inclusion, however there is one small issue...

In install.rdf you have minVersion as 26.9. As this add-on is made for 27.1.0 and above, you should change the minVersion to reflect that.
I have tested it on 27.0 and it works fine. The problem is Pale Moon refuses to install it on 27.* it I set the minVersion at 27.0.

As chance that there will ever be a Pale Moon 26.9 is near zero, I assume it is safe.

To be extra-safe, I have changed it to 26.99 in the next iteration. See below.

JSView Revive 2.1.5 (20170214):
+ updated the install minVersion to 26.99 to be extra safe
+ added missing product label and tooltip in toolbarpalette
+ remove the unused FF overlay file (and reference in chrome.manifest)
+ added reference to Pale Moon forum topic in install.rdf (homepageURL)

Cheers,
TMTisFree
Attachments
jsview-revive-2.1.5-pm_v27.xpi
JSView Revive 2.1.5 for Pale Moon v27
(122.89 KiB) Downloaded 42 times

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-14, 08:10

Matt A Tobin wrote:you need to send it to one of the add-ons team via private message not a post..
Sorry, I though 'PM' was mean to be Pale Moon, not private message...

Thanks,
TMTisFree

GreenGeek

Re: JSView 2.1.3 PM v27.1

Unread post by GreenGeek » 2017-02-14, 13:07

But I think LootyHoof was making the point that it needs the ".1.0" release to work. So, setting it to 27.0.9 (or 99 if you pefer) would prevent it from being installed on 27.0.3 and earlier.

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-14, 13:12

GreenGeek wrote:But I think LootyHoof was making the point that it needs the ".1.0" release to work. So, setting it to 27.0.9 (or 99 if you pefer) would prevent it from being installed on 27.0.3 and earlier.
I quote myself:
I have tested it on 27.0 and it works fine. The problem is Pale Moon refuses to install it on 27.* it I set the minVersion at 27.0.

New Tobin Paradigm

Re: JSView 2.1.3 PM v27.1

Unread post by New Tobin Paradigm » 2017-02-14, 18:48

Set it to 27.0.0 then

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-14, 21:42

Matt A Tobin wrote:Set it to 27.0.0 then
This works, thank you!

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-02-14, 22:11

JSView is now located on the Pale Moon Add-ons Site:

https://addons.palemoon.org/extensions/jsview-revive/

oTPS0

Re: JSView 2.1.3 PM v27.1

Unread post by oTPS0 » 2017-03-02, 05:29

tmtisfree wrote:Hi,

A working version of the good old JSView, a simple but powerful JS/CSS viewer for developers, revived for Pale Moon v27.1 (not tested on anything else).

Cheers,
TMTisFree
Thanks very much! :clap: :thumbup:

Do you have the source online anyplace? Else, you're welcome to contribute to/fork from https://github.com/rejuvenate/JSView, so more folks can help/benefit! :think:

User avatar
karlkracher
Fanatic
Fanatic
Posts: 129
Joined: 2015-12-05, 17:40
Location: berlin / germany

Re: JSView 2.1.3 PM v27.1

Unread post by karlkracher » 2017-03-02, 15:13

Great, thank you.

I selected the exe file of an external editor (i.e. C:\Program Files\KEDITW\KEDITW32.exe) but it only says "editor not found, please check the path" and than opens the file in a pale moon window.

The browser console says:

[Exception... "Not enough arguments [nsIWebBrowserPersist.saveURI]" nsresult: "0x80570001 (NS_ERROR_XPC_NOT_ENOUGH_ARGS)" location: "JS frame :: chrome://jsview/content/jsviewExternal.js :: jsviewExternal.openInExternalEditor :: line 96" data: no] jsviewExternal.js:111:0

GMforker

Re: JSView 2.1.3 PM v27.1

Unread post by GMforker » 2017-03-02, 16:19

karlkracher wrote:[Exception... "Not enough arguments [nsIWebBrowserPersist.saveURI]" nsresult: "0x80570001 (NS_ERROR_XPC_NOT_ENOUGH_ARGS)" location: "JS frame :: chrome://jsview/content/jsviewExternal.js :: jsviewExternal.openInExternalEditor :: line 96" data: no] jsviewExternal.js:111:0

Code: Select all

webBrowserPersist.saveURI(uri, null, null, null, null, file, null);
vs.

Code: Select all

webBrowserPersist.saveURI(uri, null, null, Components.interfaces.nsIHttpChannel.REFERRER_POLICY_NO_REFERRER_WHEN_DOWNGRADE, null, null, file, null);
See also bug #704320
Last edited by GMforker on 2017-03-02, 16:26, edited 1 time in total.

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-03-02, 16:25

GMforker wrote:
karlkracher wrote:[Exception... "Not enough arguments [nsIWebBrowserPersist.saveURI]" nsresult: "0x80570001 (NS_ERROR_XPC_NOT_ENOUGH_ARGS)" location: "JS frame :: chrome://jsview/content/jsviewExternal.js :: jsviewExternal.openInExternalEditor :: line 96" data: no] jsviewExternal.js:111:0

Code: Select all

webBrowserPersist.saveURI(uri, null, null, null, null, file, null);
vs.

Code: Select all

webBrowserPersist.saveURI(uri, null, null, Components.interfaces.nsIHttpChannel.REFERRER_POLICY_DEFAULT, null, null, file, null);
See also bug #704320
I have already corrected it, but thanks anyway.

Please be patient as I upload the new version to the Pale Moon repository.

Regards,
TMTisFree

tmtisfree

Re: JSView 2.1.3 PM v27.1

Unread post by tmtisfree » 2017-03-02, 16:45

It should be online now, thanks for the report.

JSView Revive 2.1.7 (20170303):
+ corrected a change in nsIWebBrowserPersist.saveURI() (aReferrerPolicy) introduced in FF36 preventing loading external editor
+ removed a console error about contributors
+ cosmetic changes (size columns in page info window right-justified)

Locked