CSSMenu: The ultimate CSS only PHPLayers replacement!

[numer9]

Yeah, I found it too. when clicking to deactivate the parent, the css-menu goes wild. it looses formatting...

numer9

[Caspar]

check the $disabled array in the class, it's not used for antyhing.

Hey calguy1000,

thanks for replying. Because I wasn't sure I tried both: removed $disabled array from CSSMenu.module.php, was that what you meant? Had no effect yet. Checked lib/classes/class.module.inc.php for a $disabled, but there is none.

Forgot to mention. I love PHP, but ain't got a clue about it yet...

[numer9]

Hey cas,
what version of CMS-MS (and CSSMenu) r u using? I had this problem with 0.10, but it's gone in 0.11beta1...

#9

[Caspar]

hey #9,

I'm using 1.01 (with 1.02's security-modification). Since I'm no PHP-Pro, I've preferred "officially released" versions so far. Is there anything in 0.11beta1 that could mean trouble for newbies? Oh, and I'm using CSSMenu 1.1.6.

Cas

[numer9]

hey,
i'm not a designer nor specialist for CMS MS, you'll have to wait for one of them to answer that. basically, it is not for newbies, but i'm using it with no problems at all (just pictures of default WYSWIG editor were missing, but switched back to TinyMCE and it's all working perfectly).

but anyway - that warning wasn't causing any problems neither (i my 0.10 installations), just that ugly waring. don't know what's that all about, haven't got time to debug it, but all functionality is working ok in 0.10, so maybe just wait until it'll be fixed by authors. and don't forget to upgrade to 0.10.2, maybe it'll fix your problem?

#9

[Caspar]

thanx #9,

I'll do that.

[calguy1000]

I found the problem using 0.11 beta 2....  it ignores properly items that arenot to be shown in the menu, but  displayes the children, and attaches them to the wrong parent.

[numer9]

Hey calguy1000,
it's not only 0.11beta2, it's a bug that has been reported couple of times already... But it's true, you have to take care of childer by yourself if you disable the parent

#9

[calguy1000]

so I fixed it

Anyways, here's the new code.  Hope this helps.

Code: Select all

          # Check for inactive items or items set not to show in the menu
          if (!$onecontent->Active() || !$onecontent->ShowInMenu())
          {
            # Stuff the hierarchy position into that array, so we can test for
            # children that shouldn't be showing.  Put the dot on the end
            # since it will only affect children anyway...  saves from a
            # .1 matching .11
            array_push($disabled, $onecontent->Hierarchy() . ".");
            continue;
          }
          /* new code */
          else {      
            # check for children of inactive items
            $skip = 0;
            foreach( $disabled as $dis ) {
                          if( substr( $onecontent->Hierarchy(), 0, strlen($dis) ) == $dis ) {
                $skip = 1;
                            break;
              }
            }
            if( $skip ) {
              continue;
            }
          }
          /* new code ends */

[Caspar]

He's the man! Hope my XAMPP's going to recover soon so I can test it...

[websuperman]

I am new to CMSMS and so far I am very impressed with what I have seen.  This CSS Menu is great.  I am having a slight issue, however.  The menu doesn't work in IE.  When viewing the source I see that the necessary ie hack java script has not been loaded.  Does this need to be added to the template?  I have noticed this with some other modules as well so it is almost certainly me.  I am using 10.2 Darkwood CMSMS and CSS Menu version 1.1.6.  Any clues would be greatly appreicated.

[Ted]

Added to svn.  Thanks!

[bkvernstuen]

I'm thinking of replacing the PHPLayers-menu with CSSMenu, but does it support both horizontal and vertical menus now? If so, then I don't even have to think about keeping PHPLayers any longer

[Ted]

The version in 0.11beta2 and svn supports horizontal menus.  PHPLayers has been removed from all future releases. 

[bkvernstuen]

Hey, thanks Wishy!

I should have had support for both horizontal and vertical though. I guess you're all working on it, so I'll hang on for the next version

Thanks again!