Wordpress Themes - WP Forum at BFA
There will be no more development for Atahualpa (or any other theme), and no support. Also no new registrations. I turned off the donation system. I may turn the forum to read only if it gets abused for spam. Unfortunately I have no time for the forum or the themes. Thanks a lot to the people who helped in all these years, especially Larry and of course: Paul. Take care and stay healthy -- Flynn, Atahualpa developer, Sep 2021

Wordpress Themes - WP Forum at BFA » WordPress Themes » Montezuma Theme »

[SOLVED] Suggestion: add new argument to bfa_content_nav (in 1.2.3)


  #1  
Old Jul 9, 2014, 08:25 PM
marklevine
 
41 posts · Jun 2010
Suggestion for a future release... For the second time while using Montezuma, I've had to hack the file content_nav.php to change the value passed to 'mid_size' from 5 to a smaller number. I did this to get the navigation tabs to fit in the columns allocated. No problem to hack but now my client must be cautious when upgrading.

I suggest a second optional parameter to bfa_content_nav(). The best solution would be to pass an array containing some or all the values to be passed into paginate_links. For example, I would call

Code:
bfa_content_nav('multinav1', array('mid_size' => 3));
/* except I don't think that you don't allow arrays
Alternately, expose one or more of the parameters for paginate_link, especially 'mid_size') with an order list such as

Code:
bfa_content_nav('multinav1', 3);
  #2  
Old Jul 10, 2014, 05:34 AM
juggledad's Avatar
juggledad
 
23,765 posts · Mar 2009
OSX 10.11.5 WP 4.x Atahualpa(all) Safari, Firefox, Chrome
Good idea, but have you looked to see if there is still an issue when there is a large (>11) number of pages?

With the default Montezuma theme settigs, using mid_size = 3 and having more than 10 pages causes the 'next' to go to the next line. I'm thinking that maybe we should just change the default to '2' and not add in another option.

Comments?
__________________
"Tell me and I forget, teach me and I may remember, involve me and I learn." - Benjamin Franklin
Juggledad | Forum Moderator/Support
  #3  
Old Jul 10, 2014, 10:01 AM
marklevine
 
41 posts · Jun 2010
I adjusted the font-size and padding through CSS and was able to get 'mid-size'=>3 to fit in a col8 division. It doesn't look as good as the original styling though. I'd recommend adding an argument because, in this case, one size does NOT fit all. Best regards!
  #4  
Old Jul 16, 2014, 07:56 AM
juggledad's Avatar
juggledad
 
23,765 posts · Mar 2009
OSX 10.11.5 WP 4.x Atahualpa(all) Safari, Firefox, Chrome
This has been fixed in version 1.2.3 which will make it's way into the WordPress theme review process, so it should show up in a week or so. There will be some on screen doc but basically you can use something like this:
HTML Code:
		<?php bfa_content_nav( 'mid_size=2&id=menunav1' ); ?>
__________________
"Tell me and I forget, teach me and I may remember, involve me and I learn." - Benjamin Franklin
Juggledad | Forum Moderator/Support

Bookmarks

Tags
bfa_content_nav, content_nav, request, suggestion



Similar Threads
Thread Thread Starter Forum Replies Last Post
Invalid Argument messages scampy Atahualpa 3 Wordpress theme 10 Feb 24, 2013 04:48 PM
Problem after updating to 3.6.7: Invalid argument applied DavidVanK New Versions, & Updating 7 Sep 2, 2011 03:11 PM
[SOLVED] Help required: Warning: fclose(): supplied argument is not a valid stream re markwalker Atahualpa 3 Wordpress theme 12 Apr 21, 2011 04:46 AM
[SOLVED] Errors: Invalid argument supplied for foreach() Call to undefined function skyline Atahualpa 3 Wordpress theme 1 Jun 4, 2009 01:41 AM


All times are GMT -6. The time now is 11:06 PM.


Powered by vBulletin® Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.