Button Problem

[CrisisX]

This problem has been fixed.

[Illori]

it looks like if you use $scripturl instead of /index.php?page=buttonpage you might get it to highlight correctly.

for example

Code Select Expand


// [Downloads] button
'downloads' => array(
'title' => $txt['downloads_menu'],
'href' => $scripturl . '?action=downloads',
'show' => $context['allow_downloads_view'],
'icon' => '',
),

[Arantor]

It's done off the name.

For 'buttonname' => array... to display, you either need to modify another part of the script, or use action=buttonname.

So what page, exactly, should it be on when highlighting that button?

[CrisisX]

It's done off the name.

For 'buttonname' => array... to display, you either need to modify another part of the script, or use action=buttonname.

So what page, exactly, should it be on when highlighting that button?

Yes, this seems to be the problem, how would I make this work for my URLS.

The URL issue is: http://www.crisisx.com/index.php?page=test

If you click the Test button it doesn't highlight, my code doesn't seem to work with $scripturl anyone know? Here is what I'm doing:

Code Select Expand

'test' => array(
'title' => 'Test',
'href' => $scripturl . '?page=test',
'show' => true,
'sub_buttons' => array(
),
),

[Illori]

can you attach your whole subs.php file to your next post?

[CrisisX]

can you attach your whole subs.php file to your next post?

Hmm? What's the point I showed the code I'm using for the button in my last post

[Illori]

I want to see where in the code yours is being put, which you have not provided us

[CrisisX]

I want to see where in the code yours is being put, which you have not provided us

In the $buttons array

Code Select Expand

if (($menu_buttons = cache_get_data('menu_buttons-' . implode('_', $user_info['groups']) . '-' . $user_info['language'] . '-' . empty($context['disable_sp']), $cacheTime)) === null || time() - $cacheTime <= $modSettings['settings_updated'])
{

$buttons = array(

'test' => array(
'title' => 'Test',
'href' => $scripturl . '?page=test',
'show' => true,
'sub_buttons' => array(
),
),
);

[Illori]

the code looks correct, any messages in your error log?

also does it work correctly with the default theme?

[edit] your url to the page is http://www.crisisx.com/index.php/page,test.html which leads me to believe this issue may be related to a mod you have installed

[CrisisX]

the code looks correct, any messages in your error log?

also does it work correctly with the default theme?

[edit] your url to the page is http://www.crisisx.com/index.php/page,test.html which leads me to believe this issue may be related to a mod you have installed

My error log is empty and the same thing seems to happen with the default theme. My only guess would be because I am not using action. Anyone else know?

[Illori]

list of installed mods?

[Kays]

On your test page, before SSI.php is included. Add the following:

Code Select Expand


$_GET['action'] = 'test';


[CrisisX]

On your test page, before SSI.php is included. Add the following:

Code Select Expand


$_GET['action'] = 'test';


Added it in Subs-Portal.php before:

Code Select Expand


if (SMF != 'SSI')
require_once($boarddir . '/SSI.php');

and now every page highlights this, I'm assuming I added it in the wrong area.

[Kays]

Where is your test page? How did you create it?

[CrisisX]

Where is your test page? How did you create it?

via SimplePortal

[Kays]

Ahh, just saw your edit on that other post..

Having that in Subs.php can mess things up. Try this instead.

Code Select Expand


if (isset($_GET['page']) && $_GET['page'] == 'test')
   $_GET['action'] = 'test';


[CrisisX]

Ahh, just saw your edit on that other post..

Having that in Subs.php can mess things up. Try this instead.

Code Select Expand


if (isset($_GET['page']) && $_GET['page'] == 'test')
   $_GET['action'] = 'test';


Thank-you.

[Kays]

Cool, glad that did it.