Re: HTML_FONT_SIZE_HUGE firing on html that looks ok

Bill Cole <[email protected]> Tue, 12 May 2026 13:14:39 -0400
Newsgroups gmane.mail.spam.spamassassin.general
Message-ID <[email protected]>
On 2026-05-12 at 09:53:33 UTC-0400 (Tue, 12 May 2026 15:53:33 +0200)
Matija Nalis <[email protected]>
is rumored to have said:

> Alternatively (and likely much better, now that I think of that), one
> could simply update just the rules:
>
> if (version >= 4.000000)
> body HTML_FONT_SIZE_HUGE    eval:html_range('max_size', '24', 'inf')
> endif
>
> if (version < 4.000000)
> body HTML_FONT_SIZE_HUGE    eval:html_range('max_size', '6', 'inf')
> endif

I like this better than further complicating the code determining font 
size.

I have committed a rule change in rules/20_html_tests.cf:

    108	if (version < 4.000000)
    109	  body HTML_FONT_SIZE_LARGE	eval:html_range('max_size', '5', 
'6')
    110	  describe HTML_FONT_SIZE_LARGE	HTML font size is large
    111	
    112	  body HTML_FONT_SIZE_HUGE	eval:html_range('max_size', '6', 
'inf')
    113	  describe HTML_FONT_SIZE_HUGE	HTML font size is huge
    114	endif
    115	
    116	if (version >= 4.000000)
    117	  body HTML_FONT_SIZE_LARGE	eval:html_range('max_size', '18', 
'24')
    118	  describe HTML_FONT_SIZE_LARGE	HTML font size is large
    119	
    120	  body HTML_FONT_SIZE_HUGE	eval:html_range('max_size', '24', 
'inf')
    121	  describe HTML_FONT_SIZE_HUGE	HTML font size is huge
    122	endif


# svn commit -m "Fix HTML_FONT_SIZE_HUGE to work with px (4.x) and 
web-sized (1-7) fonts. As suggested by Matija Nalis on the Users list."
Authentication realm: <https://svn.apache.org:443> ASF Committers
Password for 'billcole': ***************

Sending        rules/20_html_tests.cf
Transmitting file data .done
Committing transaction...
Committed revision 1934147.



-- 
  Bill Cole
  [email protected] or [email protected]
  (AKA @[email protected] and many *@billmail.scconsult.com 
addresses)
  Please keep discussion mailing list replies *on-list*
  Not Currently Available For Hire