Full addon review revealed this bug described by will:

However, I get the following error in the Browser Console when tagging all tabs. I think it might be the case that BookmarkPropertiesPanel._URIs has an extra null entry (in addition to all the open tabs) and the error is coming from trying to tag that null URI with PlacesUtils.tagging.tagURI(URI, tags);. Please try to fix this issue with your next update.

NS_ERROR_ILLEGAL_VALUE: 'Illegal value' when calling method: [nsITaggingService::getTagsForURI] @ chrome://browser/content/places/editBookmarkOverlay.js:423

Tested on Ubuntu 12.04, OS X 10.8.4 with Firefox 23, 24

using firefox 55, when I select bookmark all tabs with this plugin enabled, it displays the default dialog.

any help would be appreciated

Will pointed out this issue in full addon review:

Note that you reference a DOM mutation event. These are deprecated and usually not allowed. Yours is okay for because it mirrors the current Firefox code at http://mxr.mozilla.org/mozilla-release/source/browser/components/places/content/bookmarkProperties.js#337 . However, at some point in the future, if the Firefox code is updated, you may need to use a mutation observer instead as described here: https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver