DavidJCobb

About me

Developer Information
Name DavidJCobb
User since June 6, 2008
Number of add-ons developed 1 add-on
Average rating of developer's add-ons Not yet rated

Add-ons I've created

Bug 615866 Fix

Stops the browser from forcing links in App Tabs to open in new tabs.

Not yet rated
10 users

My Reviews

Greenhouse

The add-on causes lag-spikes on large pages. Possible fix... 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).

This review is for a previous version of the add-on (1.1). 

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.

This review is for a previous version of the add-on (3.5.9). 

Open Tab Count

Rated 4 out of 5 stars

Great. A recommendation, though: an option to always show the number of windows open (e.x. "3/1"). This way, the icon doesn't expand horizontally when a new window is opened -- I like my icons to be static, and I'm sure others do too.

Organize Status Bar

Rated 5 out of 5 stars

Great, absolutely great, just what I was looking for! One recommendation, though: the ability to add padding on a status bar icon would be great, as some icons often end up right next to each other, with no spacing... (Preferably different settings for left and right padding, if implemented).

This review is for a previous version of the add-on (0.6.3). 

Firebug

Rated 2 out of 5 stars

This would be five stars if not for the new (and unexplained and annoying) need to refresh to use the console, and the apparent removal of the option to disable a given panel for a given domain name. When these options are re-added, review will be changed to five stars.

This review is for a previous version of the add-on (1.4.2). 

Firebug

Rated 3 out of 5 stars

This'd be a 5 if not for the fact that I have to refresh to get the JS Console to work. I use it to debug errors on sites where I use userscripts; having to reload effectively resets the site's state, makes debugging sort of a challenge when I don't know what causes an error or what actions result in one.

This review is for a previous version of the add-on (1.4.2). 

RequestPolicy

Rated 5 out of 5 stars

Love it! Few suggestions, though:

- Thales79bp's suggestion (about blacklists)
- An easy way to convert Origins-to-Destinations policies to Destinations policies would be helpful (even if it's just a context-menu item when you right-click a policy)
- It'd be easier to enter several Destination if, when typing in a new Destination policy, hitting Enter hits "Allow" instead of "OK" -- and then returns focus to the textfield

This review is for a previous version of the add-on (0.5.3). 

Firebug

3 Stars Rated 3 out of 5 stars

Revising my review, lowering to 3 stars. You'll get a 5 once more of the major glitches (some of which can accurately be described as "catastrophic") are ironed out.

This review is for a previous version of the add-on (1.3.3). 

CacheViewer

Rated 4 out of 5 stars

A very useful plugin. It is, however, lacking in certain features... Some good new features to implement would be

1. Ability to delete multiple items at once (you can select multiple but can't delete multiple)

2. Ability to automatically delete based on searches (pruning)

3. Ability to block items from being cached?

4. Ability to search based on mime type, file extension, location (memory/disk) and size in addition to URL

This review is for a previous version of the add-on (0.4.7). 

Show Missing Images

Hmm... 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));

This review is for a previous version of the add-on (0.1).