WordPress.org

Ready to get started?Download WordPress

Forums

Better Tag Cloud
Improvements Can Be Made (3 posts)

  1. CatherineBr
    Member
    Posted 3 years ago #

    The following improvements can be made :

    1. The Settings tab is ignored in the latest versions of the plug-in, and should be removed altogether to avoid confusion.

    2. The font sizes inserted in the HTML code contain way too many decimals (ex : 4,33333333333333px). The round() function should be used :

    $tagsize = sprintf ( round( $smallest + ( ( $count - $min_count ) * $font_step ), 1 ) );

    3. When nofollow is enabled, the plug-in generates invalid HTML code, according to the W3C Validation Service. Why ? Because nofollow inserts a rel tag :

    rel="nofollow"

    but such rel tag already exists and should be removed :

    rel="tag"

    4. When nofollow is enabled, unnecessary spaces are added. Line :

    $echo_nofollow = ' rel="nofollow" ';

    should be :

    $echo_nofollow = 'rel="nofollow"';

    5. When replace is enabled, the plug-in replaces spaces with nobreak spaces, but it should also replace hyphens with nobreak hyphens :

    if ( $replace == 'Yes' ) {
    			$tag_name = preg_replace( '# #', ' ', $tag_name );
    			$tag_name = preg_replace( '#-#', '&'.'#8209;', $tag_name );
    		}

    6. Tags with accents (ex : é, î, ç, ü) are not sorted correctly. A user named "Emilio" already provided a solution on the plug-in homepage.

    http://wordpress.org/extend/plugins/nktagcloud/

  2. CatherineBr
    Member
    Posted 3 years ago #

    Concerning 6., which affects also WordPress core, I submitted today a bug report at :

    http://core.trac.wordpress.org/ticket/15652

    So, an official solution will be provided in a future release of WordPress, and from there, it can be incorporated in the plug-in.

  3. nkuttler
    Member
    Plugin Author

    Posted 3 years ago #

    Thanks for the feedback, I'll look into this asap! Btw, CatherineBr, if you are familiar with diff I'll be happy to review your patches.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags