Jump to content


Replying to Output Of Customer Pages Class Names


Post Options

  or Cancel


Topic Summary

Elise Alley

Posted 23 August 2013 - 12:17 PM

Hey Hugh,

Thanks for the suggestion.  And the great comments about Exchange!  :)

I'll be sure to pass this along to the developers.

Let us know if you have any other suggestions or questions.

Thanks,

Elise

Hugh Smith

Posted 22 August 2013 - 11:14 PM

The other thought I'll add to this thread is about the templating for Exchange. Very nice! Definitely like it so far.

There are too many WP plugin developers that fail to give control over markup to the user. What if don't want to use a ul/li tag/list? What if I want to use a div or span? You just give me the content, I'll mark it up. (Yes, default templates are good, especially for people who aren't exactly coders, but give me something to work with that I don't have to modify core files.)

With maybe the exception of the above :wink:, Exchange is not one of them.

(OK, I'm done.)

Hugh Smith

Posted 22 August 2013 - 08:03 PM

From what I can tell there isn't a template file for the /account/ page. As such, there doesn't seem to be a way to control the output for the customer account menus, as in the li class names. I've modified core Exchange files to get what I want, but implementing the following suggestion (or similar) would be awesome.

Change /ithemes-exchange/api/theme/customer.php at lines 421+ from:

$page_slug = trim( $page_slug );
$class = it_exchange_is_page( $page_slug ) ? ' class="current"' : '';
$nav .= '<li' . $class . '><a href="' . it_exchange_get_page_url( $page_slug ) . '">' . it_exchange_get_page_name( $page_slug ) . '</a>';

To:

$page_slug = trim( $page_slug );
$class = it_exchange_is_page( $page_slug ) ? ' current' : '';
$nav .= '<li class="it-exchange-customer-menu-' . $page_slug . $class . '"><a href="' . it_exchange_get_page_url( $page_slug ) . '">' . it_exchange_get_page_name( $page_slug ) . '</a>';

Thanks, have a great day!

Hugh

Review the complete topic (launches new window)