Wordpress Themes - WP Forum at BFA

Wordpress Themes - WP Forum at BFA (http://forum.bytesforall.com/index.php)
-   Page & Category Menu Bars (http://forum.bytesforall.com/forumdisplay.php?f=10)
-   -   [SOLVED] Menu bar has duplicate specification of attribute "class" (http://forum.bytesforall.com/showthread.php?t=8572)

hollidaysjohn Jul 28, 2010 06:20 AM

[SOLVED] Menu bar has duplicate specification of attribute "class"
 
I'm getting an error when I validate the html. Using 3.5.3 and WP 3.0 and the WP menu system.

On a menu box when the sub-menu indicator arrows appear to show there is a submenu, I get the follow error from the W3 validator:

Quote:

duplicate specification of attribute "class"
<li class="rMenu-expand menu-item-1008" class="menu-item menu-item-type-post_type">
Is that the preg_replace function in bfa_hor_pages.php?

lmilesw Jul 28, 2010 06:49 AM

I get HTML validation errors whether the "fix" is in or out but I also don't care too much about HTML validation. Go to any site and you will find HTML "errors".

ganphx Aug 22, 2010 04:25 PM

There is no guarantee that any browser would honor or use the second coded class.

The preg_replaces are incorrect in functions/bfa_new_wp3_menus.php:

Old Code:

Code:

        $newmenu = preg_replace("/<li id=\"(.*?)\n<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t\t\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t\t\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t\t\t\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t\t\t\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);
        $newmenu = preg_replace("/<li id=\"(.*?)\n\t\t\t\t\t\t\t\t\t<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\1\n\t\t\t\t\t\t\t\t\t <ul class=\"rMenu-ver sub-menu\">",$newmenu);

New Code:

Code:

        $newmenu = preg_replace("/<li id=\"(.*)\" class=\"(.*?)\">(.*)\n(.*)<ul class=\"sub-menu\">/i","<li class=\"rMenu-expand \\2\">\\3\n<ul class=\"rMenu-ver sub-menu\">",$newmenu);

juggledad Aug 23, 2010 06:05 AM

see http://forum.bytesforall.com/showthread.php?t=8756

ganphx Aug 23, 2010 11:45 AM

Thanks for the link...bugfix worked!


All times are GMT -6. The time now is 03:30 AM.

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