Mailing List Archive

[Bug 6760] HTML_FONT_LOW_CONTRAST doesn't handle 3 character color shorthand

Darxus <> changed:

What |Removed |Added
CC| |

--- Comment #1 from Darxus <> 2012-02-17 19:47:22 UTC ---
It looks to me like spamassassin hits HTML_FONT_LOW_CONTRAST when the
foreground and background colors are *close*, but does *nothing* when they're
the same. It triggers HTML_FONT_LOW_CONTRAST on exactly matching colors,
including the report this bug was opened for, if you add this in addition to my
last patch:

# invisibility
if (substr($fg,-6) eq substr($bg,-6)) {
+ $self->put_results(font_low_contrast => 1);
return 1;
# near-invisibility

Is this the right thing to do, or am I missing something?

Configure bugmail:
------- You are receiving this mail because: -------
You are the assignee for the bug.