wp_generate_tag_cloud() font-size generated with comma as decimal separator

[elvarsteinn]

Hello,
the wp_generate_tag_cloud() function in category-template.php was causing me problems since it returned style="font-size: 1,723828em" for example, and at least on FireFox 3.6 in MacOs the comma caused the browser to ignore the font-size attribute wholly, causing all but the largest tag (since that came down to an even 2em) to be the same size.

I'm guessing that on other php Configurations (im on a server in Germany, where comma is the correct decimal separator) a point is returned by php as the decimal separator, and hence for people running english php configurations, this wouldn't be a problem. Also i assume some browsers / configurations dont mind getting the value with a comma separator, but mine did, and it isnt a very strange browser/OS combination im running.

Bug (line 715-717 in category-template.php):

$a[] = "<a href='$tag_link' class='tag-link-$tag_id' title='" . esc_attr( $topic_count_text_callback( $real_count ) ) . "' style='font-size: " .

( $smallest + ( ( $count - $min_count ) * $font_step ) )
. "$unit;'>$tag_name</a>";

Fix:

$a[] = "<a href='$tag_link' class='tag-link-$tag_id' title='" . esc_attr( $topic_count_text_callback( $real_count ) ) . "' style='font-size: " .

number_format( ( $smallest + ( ( $count - $min_count ) * $font_step ) ), 3, '.', ',')
. "$unit;'>$tag_name</a>";

Of course the rounding value of 3 could be chosen differently, or set to 12 (? not sure) to not round at all (personally i feel you would never need more than 3 decimal places, and it does make the html look nicer). Alternatively, str_replace can replace a comma with a point.

[wojtek.szkutnik]

I think 4 is the maximum number of digits considered by some browsers after the decimal point.

Also, we don't want ',' as a thousands separator, do we?

[nacin]

Related #11727

[dd32]

As crazy as it may first sound, this is a possibility.
It'll only affect sites in which, the PHP Locale has been set to a certain language, for example, French.

For an example of how to trigger it, have a look at the final code in this PHP Manual comment: ​http://www.php.net/manual/en/language.types.float.php#92885

[SergeyBiryukov]

#19593, #19624

Fixed in [20675].