You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2012/02/17 20:47:22 UTC

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

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6760

Darxus <Da...@ChaosReigns.com> changed:

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

--- Comment #1 from Darxus <Da...@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.