Mailing List Archive

[Bug 6760] HTML_FONT_LOW_CONTRAST doesn't handle 3 character color shorthand
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6760

Darxus <Darxus@ChaosReigns.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |Darxus@ChaosReigns.com

--- Comment #1 from Darxus <Darxus@ChaosReigns.com> 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: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.