New CSS to pagelinks class

[Sakae]

(It rhymes )

Since I've started to use SMF, I wanted to remodelate the pagelinks class, from this:



To something like that (common used to paginate Wordpress pages):



And here is how I ALMOST got it:


1) Search @ Display.template.php for (there's 2 matches)

Code Select Expand

<div class="pagelinks

Replace with:

Code Select Expand

<div class="pagelinks2


2) Search @ Display.php for (1 match only):

Code Select Expand

$context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '[<strong>' . $txt['all'] . '</strong>] ';

Replace with:

Code Select Expand

$context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '<span class="current"><strong>' . $txt['all'] . '</strong></span> ';

3) Add this @ your Theme's css/index.css:

Code Select Expand

.pagelinks2 a, .pagelinks2 a:link, .pagelinks2 a:active {
    padding: 2px 4px 2px 4px;
    margin: 20px 0 20px 0;
    text-decoration: none;
    border: 1px solid #87845E;
    color: #333333;
    font-size: 12px;
    font-weight: bold;
    background: url(../images/pagelinks2.png) repeat-x;
}
.pagelinks2 a:hover {   
    border: 1px solid #333333;
    color: #000000;
    font-size: 12px;
    font-weight: bold;
    background-color: #FFFFFF;
    background: transparent;
}

.pagelinks2 span.current {
    padding: 2px 4px 2px 4px;
    margin: 2px;
    font-size: 12px;
    font-weight: bold;
    border: 1px solid #1C1E1C;
    color: #1C1E1C;
    background-color: white;
}
.pagelinks2 {
    padding: 2px 4px 2px 4px;
    margin: 2px;
    font-size: 12px;
    font-weight: bold;
    color: #1C1E1C;
}
#pagelinks2
{
    padding-top: 3em;
}

4) Upload the file pagelinks2.png to your theme's image folder (your theme/images/)

Working almost pretty nice now:



The PROBLEM: I can't put the CSS on the CURRENT PAGE.

Anyone can help me?

[Deaks]

what do you mean you cant get it on current page?

[Sakae]

I mean, if I am reading page 1, the new CSS I put don't work, it stays (1) (with brackets instead of () - see last screenshot).

[Deaks]

post your Display.template.php please

[Sakae]

It's the default theme, with some modifications (none of them modified pagelinks, indeed)

Hum... Attached (The message exceeds the maximum allowed length (65530 characters).)

[Hj Ahmad Rasyid Hj Ismail]

2) Search @ Display.php for (1 match only):

Code Select Expand

$context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '[<strong>' . $txt['all'] . '</strong>] ';

Replace with:

Code Select Expand

$context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '<span class="current"><strong>' . $txt['all'] . '</strong></span> ';

Can you check whether this change has really been made (and provide your Display.php here). Based on your display page, that changes hasn't been made as yet? By the way, I don't think you need the span tag. Try to remove it and make it simply:

Code Select Expand

$context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '<strong>' . $txt['all'] . '</strong> ';
See how it goes and update here will ya...

[Sakae]

Hi ahrasis

This Display.php changes the pagelink "All", which appears when you allow to see ALL answers of the topic in the same page.

Example: http://www.tigrelog.com.br/index.php?topic=6471.0;all (note the ;all in the end)

This is the only one which works, that's why the span class="current" is really necessary (see screenshot).

The question: I searched Display.php where to change the page numbers, but couldn't find.

Here's the whole part:

Code Select Expand

    // If they are viewing all the posts, show all the posts, otherwise limit the number.
    if ($can_show_all)
    {
        if (isset($_REQUEST['all']))
        {
            // No limit! (actually, there is a limit, but...)
            $context['messages_per_page'] = -1;
            $context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '<span class="current"><strong>' . $txt['all'] . '</strong></span> ';

            // Set start back to 0...
            $_REQUEST['start'] = 0;
        }
        // They aren't using it, but the *option* is there, at least.
        else
            $context['page_index'] .= '&nbsp;<a href="' . $scripturl . '?topic=' . $topic . '.0;all">' . $txt['all'] . '</a> ';
    }

[Hj Ahmad Rasyid Hj Ismail]

In your OP the problem was the first page has not followed the style/css that you assigned to it. It appears [1] instead of what you want. It is from this code:

Code Select Expand

$context['page_index'] .= empty($modSettings['compactTopicPagesEnable']) ? '<strong>' . $txt['all'] . '</strong> ' : '[<strong>' . $txt['all'] . '</strong>] ';
Notice that in [<strong>' . $txt['all'] . '</strong>], "[" and "]" is the one that creates [1]. If you have change that correctly, then there could be something else that caused that to happen.

Secondly, it is up to you to put the span tag. I don't think that it is necessary to make it work but as you have highlighted, you want it to be different, so yes, let it be there. No problem at all. If I were you, I will focus on what creates [1] other then what you have changed.

[Sakae]

Thanks for your answer ahrausis, but I insist: I changed the code I wrote on OP and still have problems with it.

Note that the $txt['all'] is not the one that creates [1], but indeed creates [All], when you allow all pages to be seeing. I attach my Display.php to see that I'm saying exactly what it's happening.

Also, what I really want is to change the [1], as you said. I just can't do it!!!

Am I missing something here?

[Hj Ahmad Rasyid Hj Ismail]

Then it must elsewhere and not in Display.php. But I am not sure where. This code $context['page_index'] is the one that is called to provide number of pages so it is a good keyword to start your research. Good luck!

[Sakae]

LOL, that's exactly what I wanted since the original post...............

[Mick.]

Sakae, did you figured it out?  Im curious as im digging this.

[Sakae]

Sakae, did you figured it out?  Im curious as im digging this.

Not yet... I'm stuck on Display.php, have no clue neither from where to begin