[glossary_term_list] shortcode bugs

  • dcmg

    (@dcmg)


    12 years, 10 months ago

    First of all, I love this plugin and I would congratulate you for your great work.

    I’m currently using version 2.4.1, and I got some troubles using [glossary_term_list] shortcode :
    #1. the <span class="glossary-list-details"> closes with </ul> !
    #2. the code doesn’t generate the expected number of columns (seems to generate columns containing N elements, instead of N columns)

    So, I made some corrections in class/wpg-shortcode-glossary-list.class.php that I would appreciate you validate and report in future versions :

    #1. replace at line 97 </ul> by </span>
    #2. add else: $cols = (count($termlist) + $cols - 1)/$cols; after line 87

    Many thanks in advance.

    http://wordpress.org/extend/plugins/wp-glossary/

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author TCBarrett

    (@tcbarrett)

    12 years, 10 months ago

    Hi dcmg

    Have you got an example page showing the issue? Or walk me through an exact way to replicate the problem?

    Thanks.

    Thread Starter dcmg

    (@dcmg)

    12 years, 10 months ago

    Hi Tom,
    I have no example page, but the issue could be reproduced easily.
    My glossary page is empty and use a template page that makes a do_shortcode('[glossary_term_list cols=3 /]').
    I only have 10 terms and as a result I got 4 columns containing 3 terms, except the last one (1 term). Reading the source code makes me consider there was a bug and I corrected it.
    Another solution could be to use the CSS to generate multicolumns…
    -Damien

Viewing 2 replies - 1 through 2 (of 2 total)

The topic ‘[glossary_term_list] shortcode bugs’ is closed to new replies.