A bunch of Mozilla/FF incompatibilities (with workarounds)

[flexo]

I thought I'd list a few things tb.js doesn't properly wrap for Mozilla / FF, so Softomate can either fix it or other users know how to work around the shortcomings:

In MSIE you can change the image of a button using:

tool.SetPropertyById("foo", "img", "23");

This currently doesn't work in Firefox. Instead you can do:

tool.SetPropertyById("foo", "image", "chrome://toolbar/content/23.png");

Note that the first command in Firefox is silently ignored (so to be upwards compatible I suggest you just issue both). Fixing this should be easy for Softomate.

Second, tool.Var(1) = 23; doesn't work in Firefox - Instead you have to do tool.Var[42] = 23;

Note that the first command throws an Exception so to be portable you should do something like

try { tool.Var(1) = 23; } catch(e) { tool.Var[42] = 23; }

As for Softomate - also easy to fix, the Mozilla JavaScript engine supports getters / setters using the methods __defineSetter__ / __defineGetter__. Just make Var a method returning an object with a getter & setter function.

Third, context menus don't work in Mozilla. No easy fix here as the XML format differs too much (I'm no XSLT wiz). A workaround is to add your own <menuitem> to tb.xul after converting your toolbar - I wrote a bunch of batch scripts + a ruby script which extract the jar, modify the XUL & JS file and compress it again.

Just add this in the <popup>

<menuitem id="some_id" label="The Label"
oncommand="some_function(gContextMenu);"/>

And to tb.js you add:

function some_function(contmenu)
{
try {
var term = "";
try {
term = contmenu.searchSelected();
} catch(k) {
term = getBrowserSelection();
}
// do something with "term" now...
} catch(e) { }
}

Fourth "BeforeNavigate" scripts don't seem to work in FF. I hacked around this by adding this to tb.js:

var old_fn = toolbar_refresh_banner;

toolbar_refresh_banner = function(str) {
old_fn(str);
if(str == "BeforeNavigate") {
// Your code...
}
}

Okay. This got ugly enough for now. HTH & Cheers

Felix