@Paulo No worries man, I was attempting to update my code in 0.2pre13 and then I figured I would merge yours. Mine is apparently a bit out of date since I was modifying an old version.
Topher On Wed, Feb 18, 2009 at 11:22 AM, psargaco <[email protected]> wrote: > > I've tested my command and corrected some bugs, so I'm posting the new > version on Gist, following Fernando's suggestion. You can find the > command here: > > http://gist.github.com/66417 > > I noticed that the original built-in command has two bugs. When the > modifier "to" is tested for content, a simple > > if(modifiers.to) > ... > > won't do. This resulted in the word "to" always being included in the > preview. I changed it to > > if(modifiers.to.text.length > 0) > ... > > I intended to report this bug on Trac, but as I posted earlier, Trac > seems to be indisposed. > > @Toph: Sorry, haven't integrated your ideas yet. > > > -- Topher Fangio Software Developer [email protected] --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "ubiquity-firefox" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/ubiquity-firefox?hl=en -~----------~----~----~----~------~----~------~--~---
