eyalroz

About me

Developer Information
Name eyalroz
Location Haifa
Occupation recently fired...
Homepage http://earendil.ath.cx/
User since March 5, 2007
Number of add-ons developed 2 add-ons
Average rating of developer's add-ons Rated 4 out of 5 stars

In a little more detail...

I'm about to complete my Ph.D. in theoretical Computer Science, and struggling to make ends meet...

Add-ons I've created

Remove Duplicate Messages (Alternate)

Search and remove messages which are the same as other messages in your mail folders (referred to as duplicates or dupes).

Rated 4 out of 5 stars (131)
66,215 users

BiDi Mail UI

Direction control and BiDi-related charset misdetection correction

Rated 4 out of 5 stars (24)
2,952 users

My Reviews

Remove Duplicate Messages

Use the better alternative to this extension! Rated 1 out of 5 stars

This extension has been effectively abandoned by its author for almost 2 years, without significant updates. It is also not free software, so the code cannot be used elsewhere.

For this reason I have fully rewritten a dupe removal extension of my own, which is actually free software:

Remove Duplicate Messages (alternate)
https://addons.mozilla.org/en-US/thunderbird/addon/4654

It has seen more active development, has a richer feature set (which proerly includes the features of this extension) and for which issues can be opened on Mozdev's bugzilla and addressed within a reasonable amount of time.

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

Facebook Hebrew RTL

This is a nice Greasemonkey script, not an extension Rated 2 out of 5 stars

The extension works as claimed. However, it is basically a Greasemonkey script, coupled with a copy of some of the GM code to make it work independently. I do not see a reason why this kind of a GM script should become its own extension. Author is free to do as he pleases but I do not believe this should be supported by making this extension public.

Minor nits:

- Regular expression used to detect Hebrew should be more tersely and clearly constructed.

- A single letter should probably not suffice to RTL an entire paragraph or line; try an entire word, or at least a sequence of characters

- Poor coding style in various places, such as defining a function named '$', adding to the regexp prototype a method named 'loc', testing for 7 regexps instead of testing for their '|'ification, etc.

- If one uses an independent extension rather than a Greasemonkey script, why keep the images used as "data:/..." strings rather than as chrome files (or a single file)?

- Empty chrome/ directory and a content/ directory outside it

- Use of spaces and tab characters inconsistently