Jump to content


information

Output Of Customer Pages Class Names



2 replies to this topic

#1 Hugh Smith

Hugh Smith

    Advanced Member

  • Members
  • Others: Toolkit
  • PipPipPip
  • 95 posts

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

#2 Hugh Smith

Hugh Smith

    Advanced Member

  • Members
  • Others: Toolkit
  • PipPipPip
  • 95 posts

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.)

#3 Elise Alley

Elise Alley

    iThemes Support

  • Administrators
  • Others: All Plugins, All Themes, Members, Moderators, Toolkit, Webdesign
  • 5,120 posts
  • LocationOklahoma City, OK

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



Reply to this Topic


ATTENTION
If this topic is marked RESOLVED or INFORMATION, or if you are NOT experiencing EXACTLY the same issue, please start a new topic to provide the details of your problem. If the solution provided doesn't work for you, you should start a new topic. If you feel this thread contains relevant information, you may include a link to it in your new topic. To ensure that we can process your support request efficiently, ALWAYS include a link to your site, and/or the page your request is related to. When posting your question or request on the forum, please be as concise and specific as possible. The shorter and more to the point you can make your request, the better.

For BackupBuddy related issues, be sure to include the status log from your most recent backup.

CLICK HERE to start a new topic in the Exchange ecommerce plugin forum.


  



Also tagged with one or more of these keywords: information

0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users