ppi / website

The PPI Website
http://www.ppi.io
MIT License
23 stars 15 forks source link

Topbar menu active status #25

Closed alfrekjv closed 12 years ago

alfrekjv commented 12 years ago

The topbar Menu needs to get implemented the active status, so users/visitors know where they are at a simple glance.

Currently there's an .active class (and it's attached to Home), it just need to be integrated on the code.

Any volunteers? lol

Thanks a lot.

dragoonis commented 12 years ago

@alfrekjv

You have access to the $request variable in ALL ppi views,

$request['controller'] and $request['method']

with this you can make ternaries for the main nav.

<a href="" class="<?= $request['controller'] == 'something' ? 'active' : ''; ?>" >Something</a>
alfrekjv commented 12 years ago

Kewl, let me add that feature to close this ticket.

dragoonis commented 12 years ago

@alfrekjv add a PR, thanks! :-)

alfrekjv commented 12 years ago

Problem Solved.

dragoonis commented 12 years ago

Thanks for the commit @alfrekjv