happy_melon wrote:

The code is at the bottom of /includes/RecentChange.php - I'd just replace the stacked ifs with something like this:

return '<' 
       . ( abs($szdiff) > abs($wgRCChangedSizeThreshold) ? 'strong' : 'span' ) 
       . ' class="mw-plusminus-'
       . ( $szdiff == 0 ? 'null' : ( $szdiff > 0 ? 'pos' : 'neg' ))
       . '/>(' . ( $szdiff > 0 ? '+' : '' ) 
       . $formatedSize . ')</span>';

Backwards compatible AFAIK.

happy_melon wrote:

Patch

attachment bug15925.txt ignored as obsolete

Patch applied in r42078

Patch appears to have completely messed up RecentChanges. The +- character parts no longer appear, and the majority of Special:RecentChanges became bolded.

Fixes bug while reverting previous commit (to RecentChange.php) related to this.

attachment bug15925.diff ignored as obsolete

Reverted r42078 in r42088 -- bad HTML completely breaks output, as well as just making the code much harder to read.

Improved per suggestions by Brion on MediaWiki-General.

attachment bug15925v2.diff ignored as obsolete

More improvements

attachment bug15925v3.diff ignored as obsolete

Looks good!

happy_melon wrote:

<cries in corner>Methinks I need my own copy of MediaWiki that actually runs... Such a stupid error too... </cries in corner>

Sorry for breaking the universe :-D

Only thing I wonder, why are we bothering with a new global? Wouldn't it be easier to just abs() like in the original patch?

Uses abs function instead of new global per Demon.

attachment bug15925v4.patch ignored as obsolete

happy_melon wrote:

Wouldn't using abs() on $szdiff as well obviate the need for the more complex logic test?

^demon already did it to his local copy but said nothing here.

use abs more in check, use .patch format.

attachment bug15925v5.patch ignored as obsolete

Applied correctly and with all the fixes in r42110

alexsm333 wrote:

There was a good reason why additions weren't bold: discussion pages.

With r42110 both RC and watchlist look really ugly. There are so many bold green numbers that the boldness simply becomes useless.

happy_melon wrote:

the bytecounts can be styled easily enough;

strong.mw-plusminus-pos {font-weight:normal;}

would do the trick. Styling can be applied at either an individual, wiki, or sitewide level. The important thing is that the facility is there to have the numbers bolded if desired.