Add Bookmark Here ² not working?

This board is for discussions, bug reports, etc. for pre-releases of the v27 milestone codenamed "Tycho".

Since the beta phase is over, this board is closed for new posts/topics.

Moderator: satrow

User avatar
kizo07
Banned user
Banned user
Posts: 344
Joined: Mon, 17 Oct 2016, 14:11

Add Bookmark Here ² not working?

Unread postby kizo07 » Fri, 28 Oct 2016, 00:55

It seems that Add Bookmark Here ² (Version 23.0.20140225.1) not working in 27b3

User avatar
Shadeclan
Lunatic
Lunatic
Posts: 274
Joined: Thu, 05 Jun 2014, 17:27
Location: Albany, NY

Re: Add Bookmark Here ² not working?

Unread postby Shadeclan » Fri, 28 Oct 2016, 16:50

I verified this using the latest version of this add-on that still works with Pale Moon - 23.0.20140225.1
Image

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 22116
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: Add Bookmark Here ² not working?

Unread postby Moonchild » Sat, 29 Oct 2016, 09:00

The extension will have to be updated to work with the changes to the places API. Very likely it will have to target Pale Moon specifically from this point forward.

Code: Select all

Error: TypeError: PlacesUtils.nodeIsReadOnly is not a function
Source File: chrome://abhere2/content/browserOverlay.js
Line: 480
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne

Image

User avatar
kizo07
Banned user
Banned user
Posts: 344
Joined: Mon, 17 Oct 2016, 14:11

Re: Add Bookmark Here ² not working?

Unread postby kizo07 » Sat, 29 Oct 2016, 19:40

Thanks for your reply. It was nice add-on for the easiest and fastest bookmarking. I know that is not a priority but it should be considered to be finally interpreted/coded into the future PM .. just like in Opera or Maxthon. Something like 'Bookmark Current Page' or 'Add Here' in every bookmarks folder.
Otherwise I have for years been a user of Maxthon, until ver.4. After that 'fox' for a few months and the last two years I use PM. The best browser to date.. and I already see that PM 27 gonna be even better. Thanks again.

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1509
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: Add Bookmark Here ² not working?

Unread postby JustOff » Sun, 30 Oct 2016, 10:26

Moonchild wrote:The extension will have to be updated to work with the changes to the places API.

The fix for PM27+ is simple (actually taken from abh2 31.0.20150313):

Code: Select all

--- browserOverlay.js.orig   Mon Feb 24 01:28:10 2014
+++ browserOverlay.js   Sun Oct 30 12:23:11 2016
@@ -299,0 +300,4 @@
+   nodeIsReadOnly: function(node) {
+      return PlacesUtils.nodeIsFolder(node) && PlacesUIUtils.isContentsReadOnly(node); // Firefox36+
+   },
+
@@ -355 +359 @@
-      if (PlacesUtils.nodeIsReadOnly(node)) return;
+      if (abHere2.nodeIsReadOnly(node)) return;
@@ -480 +484 @@
-         if (ip && ip.node && !PlacesUtils.nodeIsReadOnly(ip.node)) {
+         if (ip && ip.node && !abHere2.nodeIsReadOnly(ip.node)) {

I can share patched xpi (verified, works for me), but I'm not sure how to do it in most acceptable way.
Here are the add-ons I made in a spare time. That was fun!

User avatar
New Tobin Paradigm
Knows the dark side
Knows the dark side
Posts: 4623
Joined: Tue, 09 Oct 2012, 19:37

Re: Add Bookmark Here ² not working?

Unread postby New Tobin Paradigm » Sun, 30 Oct 2016, 11:10

We do have an Add-ons Site for a reason.

In fact we are prepping a major update to the Add-ons Site with all new software written by my self over the past week to make somewhat easier for everyone and to get a few things that have been requested.

In any event, you can always rebrand, change the author and id, and make it a fork. Unless you can get upstream to accept a patch. Forking is likely the better option as long as the license allows it.

User avatar
Shadeclan
Lunatic
Lunatic
Posts: 274
Joined: Thu, 05 Jun 2014, 17:27
Location: Albany, NY

Re: Add Bookmark Here ² not working?

Unread postby Shadeclan » Mon, 31 Oct 2016, 17:48

Sounds like a plan. Why not make a fork - especially if you already have the fix.
Image

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1509
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: Add Bookmark Here ² not working?

Unread postby JustOff » Fri, 04 Nov 2016, 18:45

Until I figured out with addons.palemoon.org, here is the fix.
Here are the add-ons I made in a spare time. That was fun!

User avatar
Doelli
Apollo supporter
Apollo supporter
Posts: 39
Joined: Sat, 11 Oct 2014, 15:25
Location: Germany

Re: Add Bookmark Here ² not working?

Unread postby Doelli » Sat, 05 Nov 2016, 09:11

This seems like almost the same issue like in this post:
viewtopic.php?p=94355#p94355

Does anyone know, how to fix it (without removing code from "AddBookmarkHere.uc.js ", as i did)?

User avatar
kizo07
Banned user
Banned user
Posts: 344
Joined: Mon, 17 Oct 2016, 14:11

Re: Add Bookmark Here ² not working?

Unread postby kizo07 » Sun, 06 Nov 2016, 01:47

Works flawlessly! Many thanks JustOff


Return to “Pale Moon "Tycho" alpha/beta”

Who is online

Users browsing this forum: No registered users and 1 guest