userChrome.js/addbookmarkhere

[Doelli]

I noticed that the AddBookmarkHere.uc.js for the AddOn userChrome.JS doesn´t work in PM 27 Beta 1.
Is this the known issue #554?

[Moonchild]

Issue #554 relates to RSS/Atom feeds. Unless that script tries to hook into that to create a bookmark (which would be strange...) it is completely irrelevant.

[Doelli]

Hi Moonchild,

thank you for your quick answer!
Is it possible to fix this issue with the userChrome.js by yourself or should i contact the Developer of the script?

[Doelli]

Hi Moonchild,

i´d fixed the Scrip byself by removing this Lines in the "AddBookmarkHere.uc.js":

Code: Select all

            if (selectedNode) {
                if (PlacesUtils.nodeIsFolder(selectedNode) /* Firefox 21+ 不兼容 && !PlacesUtils.nodeIsLivemarkContainer(selectedNode) */ && !PlacesUtils.isReadonlyFolder(selectedNode)) {
                } else {
                    iid = bookmarks.getFolderIdForItem(selectedNode.itemId);
                    var id = bookmarks.getItemIndex(selectedNode.itemId);
                    aid = e.shiftKey ? id : id + 1;
                }
            } else {
                iid = view.result.root.folderItemId;
                aid = e.shiftKey ? 0 : bookmarks.DEFAULT_INDEX;
            }

No the script works fine!
Maybe you can tell me the reason why and what´s different to PM 26.x

Thanks!

[Doelli]

Unfortunately, the script works only unreliable.
I will make a request under the "Browser extensions".

Thank you!