Add-ons
DavidJCobb
About me
| Name | DavidJCobb |
|---|---|
| User since | June 6, 2008 |
| Number of add-ons developed | 0 add-ons |
| Average rating of developer's add-ons | Not yet rated |
My Reviews
Stylish - Custom themes for any website
Rated 2 out of 5 stars
This add-on now works on Firefox for Android, but only if you're a web developer with a USB cable. I am, so I'm changing my review from 1 star to 2.
The settings page is now accessible on mobile, though it's also very cumbersome to use there. Android users can now edit userstyles on their phones, but it still isn't possible to install styles from Userstyles.org: the button takes you to the Google Chrome add-on listing. (Well done.)
It's no longer possible to install styles from arbitrary URLs, either, so the only way to get any userstyles onto your device is to use USB debugging to insert the code into Stylish's editor textbox. Stylish uses a rich text editor, so you can't just paste your code into a textarea in the DOM; instead, you need to focus an editor and then run {getEditorInSight().getDoc().setValue("your CSS here");} with your style appropriately escaped. Irritating.
The "Import/Export Firefox" feature is poorly explained. I can paste Firefox-oriented styles into the main editor without using it and they appear to work, without the syntax checker complaining. (The syntax checker does complain endlessly about CSS variables, though: it wrongly reports them as errors.)
I also found what look like analytics tracking calls (I'm guessing Google?) inside of the manage styles page as well. These calls seem to track UI events, like editor modals opening. Didn't look like they actually sent *out* the content of any userstyles being edited, but I didn't look too closely.
And lastly, styles are applied with a delay. If you make a night mode style for some website, you may still see flashes of white while the page loads.
Another five-star add-on turned into an unusable two-star mess with the launch of Firefox 57.
Stylish - Custom themes for any website
Rated 1 out of 5 stars
Well, my mobile installation just auto-updated to the WebExtension version 3.0, and now all my styles are gone and the add-on is totally unusable. Great work with WebExtensions, Mozilla! :D
This review is for a previous version of the add-on (2.1.1). This user has other reviews of this add-on.Stylish - Custom themes for any website
Rated 4 out of 5 stars
Works perfectly on desktop. However, styles can't be edited locally on mobile; they must be downloaded from the web, which is a pain for private-use, personal, and experimental styles.
This review is for a previous version of the add-on (2.0.7).Clean Links
Rated 2 out of 5 stars
The add-on's basic functionality works, but it's buggy and a pain to configure, and it seems abandoned.
The option to prevent the add-on from tampering with links to certain domains doesn't work reliably. In some cases, Clean Links breaks hyperlinks even when those links don't contain any targeted patterns, link to an excluded domain, and the excluded domain is part of the exclusion regex. This can be verified by disabling the add-on and refreshing the affected page.
Moreover, to change the regexes or domain lists, you have to use a one-line textbox that doesn't disable autocorrect or autocomplete, which is an enormous pain on mobile. The regexes textbox is also badly implemented, popping warnings as you type certain characters because regexes shouldn't end with certain characters (why are you verifying onkeypress and not onchange?).
Greenhouse
Rated 3 out of 5 stars
You can optimize your calls to the jQuery highlight function, because it also accepts an array of multiple names, handling all of them during a single search of the document.
Accordingly, instead of having "script.js" call $("p").highlight(...) in a loop (one call per name), you should gather all of the politicians' names into a single array and pass that to the highlight function. This will considerably reduce the number of times you search through the DOM.
I suspect there may be other tricks you could employ (e.g. asynchronicity, placing your code in single-millisecond timeouts to avoid blocking).
FoxyTunes
Rated 4 out of 5 stars
Great add-on. I don't actually have speakers -- I've installed it as preparation -- so I'm basing my current feedback on its integration with the GUI.
Suggestions (already submitted via feedback on your site, but I figured I'd add 'em here):
1. Ability to change certain controls. Honestly, I have no use for anything but the Media Player picker, the Open File button (which, strangely, doesn't show up), and the Play/Pause button.
2. Ability to hide the show/hide arrows on the sides.
3. An options dialog -- the context menu system is cumbersome and annoying, as it must be re-opened and re-navigated after each individual setting change. A dialog would have the added advantage of being able to hide the options button on the toolbar itself.
Show Missing Images
Rated 3 out of 5 stars
It's good. It works. It's not just a cheap SRC-changer -- it actually works just like a normal 404 placeholder should.
However, it is sort of rough and slightly unpolished. Whereas a normal 404 placeholder is shown with a border around it*, there is no border here, making it look kind of rough. Plus, broken images are only converted to placeholders after everything has loaded. Perhaps an onError event handler can be attached to images right as they start to load, so that when they fail they turn into 404 placeholders instantly?
* To see exactly what a placeholder would look like, try heading to Google.com and using this bookmarklet -- it'll take out the first image and let you see a 404 in detail.
javascript:alert(document.images[0].src="jhgajhfgafdg");
This next bookmarklet adds a 404 placeholder without height and width set, if you want to see what that would look like:
javascript:a=document.createElement("img");a.src="jhfahfdg";alert(document.body.appendChild(a));
To create your own collections, you must have a Mozilla Add-ons account.
