Font Anti-Aliasing customization

[spiff]

Hi All!

I've successfully developed Font Anti-Aliasing customization feature in muCommander (from HEAD revision). I've found ticket (http://trac.mucommander.com/ticket/159) about it and decided to help muCommander community.

Also I've developed supporting of two languages for this feature - RU and EN.

Attached you will find my patch and full-size screenshot. I hope my code will be in muCommander trunk

Thank you for muCommander!

[arik]

Hey,

Thank you for your contribution!

I'll check it on the weekend and then reply back.

Regards,
Arik

[spiff]

Thanks a lot!

But, please use my cleared version of patch (tabs was replaced by spaces, unused imports was deleted). See attached "cleared_aa.patch".

[arik]

Hey,

Here's my feedback:
1. It seems that the system property "awt.useSystemAAFontSettings" was introduced at java 1.6 (See http://docs.oracle.com/javase/6/docs/te ... ml#aaFonts). muCommander still supports java 1.5 that refered to the system property "swing.aatext" instead - we should distinguish this case also.
2. I read at http://mindprod.com/jgloss/antialiasing.html that there are more values other than "lcd" such as "gasp". I think we should let the user choose between those options if we can't determine what is the best option for him.
3. I think a better way would be to add two more attributes for MuConfiguration (which was replaced yesterday by MuPreferences class by the way):
FONT_ANTIALIASING_ON = ...
FONT_ANTIALIASING_OFF = "off"
and then DEFAULT_FONT_ANTIALIASING will be changed to DEFAULT_FONT_ANTIALIASING = FONT_ANTIALIASING_ON and
String aa = isSelected() ? MuConfiguration.DEFAULT_FONT_ANTIALIASING : "off";
will be changed to
String aa = isSelected() ? MuConfiguration.DEFAULT_FONT_ANTIALIASING : MuConfiguration.FONT_ANTIALIASING_OFF;
4. Most of the properties under the preferences dialog are changed when choosing "apply" or "ok" options. that's not the case with the anti-aliasing checkbox - so we may want to add a "restart required to apply the change" next to it, or trying to refresh the mainframe with the updated configuration if it's possible.

Thanks again,

Regards,
Arik