Buying Need help with php function - simple change

I have a function which outputs the navigation panel in a quiz module for one of my sites. Problem is I need to change the layout of the nav panel. The code is below. You can see that links to pages are output in the nav panel. However, when there's a lot of pages, they stretch all the way off the right side of the screen and the user has to scroll to the right to see them all. It looks unprofessional.

I need to have a fix where the pages, if more than say 20, are output (wrapped) on multiple rows in the nav panel.

Who can help for $10? Payment by Paypal when work is finished.

function quiz_print_navigation_panel($page, $pages) {
    //$page++;
    echo '<div class="paging pagingbar">';
    echo '<span class="title">' . get_string('page') . ':</span>&nbsp;';
    if ($page > 0) {
        // Print previous link
        $strprev = get_string('previous');
        echo '&nbsp;<a class="previous" href="javascript:navigate(' . ($page - 1) . ');" title="'
         . $strprev . '">(' . $strprev . ')</a>&nbsp;';
    }
    for ($i = 0; $i < $pages; $i++) {
        if ($i == $page) {
            echo '&nbsp;<span class="thispage">'.($i+1).'</span>&nbsp;';
        } else {
            echo '&nbsp;<a href="javascript:navigate(' . ($i) . ');">'.($i+1).'</a>&nbsp;';
        }
    }

    if ($page < $pages - 1) {
        // Print next link
        $strnext = get_string('next');
        echo '&nbsp;<a class="next" href="javascript:navigate(' . ($page + 1) . ');" title="'
         . $strnext . '">(' . $strnext . ')</a>&nbsp;';
    }
    echo '</div>';
}

Code (markup):

 

Try:

function quiz_print_navigation_panel($page, $pages) {
    //$page++;
    
    if($pages < 20){
    echo '<div class="paging pagingbar">';
    echo '<span class="title">' . get_string('page') . ':</span>&nbsp;';
    if ($page > 0) {
        // Print previous link
        $strprev = get_string('previous');
        echo '&nbsp;<a class="previous" href="javascript:navigate(' . ($page - 1) . ');" title="'
         . $strprev . '">(' . $strprev . ')</a>&nbsp;';
    }
    for ($i = 0; $i < $pages; $i++) {
        if ($i == $page) {
            echo '&nbsp;<span class="thispage">'.($i+1).'</span>&nbsp;';
        } else {
            echo '&nbsp;<a href="javascript:navigate(' . ($i) . ');">'.($i+1).'</a>&nbsp;';
        }
    }

    if ($page < $pages - 1) {
        // Print next link
        $strnext = get_string('next');
        echo '&nbsp;<a class="next" href="javascript:navigate(' . ($page + 1) . ');" title="'
         . $strnext . '">(' . $strnext . ')</a>&nbsp;';
    }
    echo '</div>';
    } else {
    
        echo '<div class="paging pagingbar">';
    echo '<span class="title">' . get_string('page') . ':</span>&nbsp;';
    if ($page > 0) {
        // Print previous link
        $strprev = get_string('previous');
        echo '&nbsp;<a class="previous" href="javascript:navigate(' . ($page - 1) . ');" title="'
         . $strprev . '">(' . $strprev . ')</a>&nbsp;<br>';
    }
    for ($i = 0; $i < $pages; $i++) {
        if ($i == $page) {
            echo '&nbsp;<span class="thispage">'.($i+1).'</span>&nbsp;';
        } else {
            echo '&nbsp;<a href="javascript:navigate(' . ($i) . ');">'.($i+1).'</a>&nbsp;<br>';
        }
    }

    if ($page < $pages - 1) {
        // Print next link
        $strnext = get_string('next');
        echo '&nbsp;<a class="next" href="javascript:navigate(' . ($page + 1) . ');" title="'
         . $strnext . '">(' . $strnext . ')</a>&nbsp;<br>';
    }
    echo '</div>';
    }
}

PHP:

Also would help if you could let me know your site url so i can see...

 

job is done. thanks everyone.

 

danx10, I went with an earlier fix from another person.