wet-boew-wordpress icon indicating copy to clipboard operation
wet-boew-wordpress copied to clipboard

Build Mobile Broken by Secondary Menu

Open svmillin opened this issue 11 years ago • 6 comments

Just looking for a bit of input as to why, after updating from WET 3.0.3 (Build: 2012-12-21 06:59 PM) to WET 3.0.6 (Build: 2013-04-11 10:57 AM), mobile view seems to be broken by the secondary menu. From what I can tell, it seems to be due to changes in pe-ap-min.js.

I tried updating to WET 3.1.3 and the issue persists. In mobile view, instead of breaking entirely, it simply outputs "undefined" underneath the secondary menu header.

The error: Uncaught TypeError: Cannot read property 'className' of undefined pe-ap-min.js:9

Thoughts?

svmillin avatar Sep 12 '13 15:09 svmillin

Yes, there's an issue with the secondary navigation not loading in mobile view for version 3.1.3 but it works fine for me in version 3.0.6 I'll look into it and see if I can fix it for 3.1.3

Thanks.

Joberrr avatar Sep 12 '13 18:09 Joberrr

If you can, try reducing the number of items in your secondary navigation via WP widgets to see if any of those items are causing the error in 3.0.6

Joberrr avatar Sep 12 '13 18:09 Joberrr

In 3.0.6, just created:

  1. A test sec-nav with a header and no items (just h3) - didn't work
  2. A test sec-nav with a header and one item - didn't work.

If I roll-back to 3.0.3, each of the test menus work. If you'd like, I can link you to the development environment (thanks, btw).

svmillin avatar Sep 12 '13 18:09 svmillin

Yeah sure, I might be able to help. As you can see it works fine here: http://www.lostprototype.com/wet-boew3/

thanks.

Joberrr avatar Sep 12 '13 19:09 Joberrr

Perfect. Yeah, that's why I was confused that it didn't work on my end. I compared your sec-nav with mine and can't spot any major differences.

svmillin avatar Sep 12 '13 19:09 svmillin

Apparently no PMs on github (shocked). Could you please send me an email @ scott . [email protected]? I'll reply back with dev access details.

svmillin avatar Sep 12 '13 19:09 svmillin