Title: Syntax Errors
Last modified: May 9, 2019
---
# Syntax Errors
* Resolved [David Elstob](https://wordpress.org/support/users/davwaiguy/)
* (@davwaiguy)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/)
* There is a syntax error in the following code:
* ```
add_theme_support(
'amp',
array(
'nav_menu_dropdown' => array(
'sub_menu_button_class' => 'dropdown-toggle',
'sub_menu_button_toggle_class' => 'toggled-on', // (optional)
'expand_text ' => __( 'expand', 'domain' ),
'collapse_text' => __( 'collapse', 'domain' ),
),
),
);
```
* The last comma needs removing please.
* On the following page, you use an almost identical solution and that also needs
the very last comma removing.
* [https://amp-wp.org/documentation/playbooks/toggling-hamburger-menus/](https://amp-wp.org/documentation/playbooks/toggling-hamburger-menus/)
* ```
add_theme_support(
'amp',
array(
'nav_menu_toggle' => array(
'nav_container_id' => 'site-navigation',
'nav_container_toggle_class' => 'toggled-on',
'menu_button_id' => 'site-navigation-toggle',
'menu_button_toggle_class' => 'toggled-on',
),
),
);
```
* Hope this helps, because I’m a non-coder, well a coding newbie to be exact, and
for people like me these things are frustrating if you don’t know how to resolve
them.
* Thanks in advance.
* David
- This topic was modified 6 years, 11 months ago by [David Elstob](https://wordpress.org/support/users/davwaiguy/).
Reason: To include code tags
* The page I need help with: _[[log in](https://login.wordpress.org/?redirect_to=https%3A%2F%2Fwordpress.org%2Fsupport%2Ftopic%2Fsyntax-errors-7%2F%3Foutput_format%3Dmd&locale=en_US)
to see the link]_
Viewing 12 replies - 1 through 12 (of 12 total)
* Plugin Author [Weston Ruter](https://wordpress.org/support/users/westonruter/)
* (@westonruter)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11515597)
* Thanks for reporting! The pages have been fixed.
* Thread Starter [David Elstob](https://wordpress.org/support/users/davwaiguy/)
* (@davwaiguy)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11519484)
* [@westonruter](https://wordpress.org/support/users/westonruter/) The code works
perfect regarding the menu, but it’s also throwing two errors in the AMP Validator
for missing navigation tags. One for tabindex and the other for role.
* I know how to fix them, but unsure about where to place the code.
* Is there any chance you can point me in the right direction please?
* [https://parallaxsite.com](https://parallaxsite.com)
* code to be introduced:
* `role="navigation" tabindex="0"`
* Cheers!
* Plugin Author [Weston Ruter](https://wordpress.org/support/users/westonruter/)
* (@westonruter)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11519538)
* [@davwaiguy](https://wordpress.org/support/users/davwaiguy/) Hey, I don’t think
the code is quite right. It should not be targeting the `nav` to toggle the menu.
It should rather be targeting the `menu-toggle` button. Please share the code
you are using.
* So you should add an `id` to this element:
* ``
* For example, `site-navigation-toggle`:
* ``
* And then you can the `menu_button_id` as shown in the example above.
* Thread Starter [David Elstob](https://wordpress.org/support/users/davwaiguy/)
* (@davwaiguy)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11519687)
* Hi Weston, I used your example from AMP-WP.Org as so…
* ```
add_theme_support(
'amp',
array(
'nav_menu_toggle' => array(
'nav_container_id' => 'site-navigation',
'nav_container_toggle_class' => 'toggled',
'menu_button_id' => 'site-navigation',
'menu_button_toggle_class' => 'toggled',
),
)
);
```
* Then that works fine, but shows the 2 errors I mentioned.
* When I filled in the blanks, the errors go away in the AMP Validator, but I just
don’t know where to place the snippets of code in the various PHP files?
* Hope this makes sense? Thanks.
* Thread Starter [David Elstob](https://wordpress.org/support/users/davwaiguy/)
* (@davwaiguy)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11519703)
* The attribute ‘role’ in tag ‘nav’ is missing or incorrect, but required by attribute‘
on’.
* The attribute ‘tabindex’ in tag ‘nav’ is missing or incorrect, but required by
attribute ‘on’.
* That’s what the errors say word for word.
* So when I type role=”navigation” and tabindex=”0″ in the AMP Validator it goes
away so I just presumed the code was correct, if I placed it in the appropriate
file?
* Plugin Author [Weston Ruter](https://wordpress.org/support/users/westonruter/)
* (@westonruter)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11519815)
* You need to add an `id` attribute to the button and use it as the value for `
menu_button_id`.
* Thread Starter [David Elstob](https://wordpress.org/support/users/davwaiguy/)
* (@davwaiguy)
* [6 years, 11 months ago](https://wordpress.org/support/topic/syntax-errors-7/#post-11520793)
* [@westonruter](https://wordpress.org/support/users/westonruter/) I understand
what you are saying, but I have done that and I am using the correct button id,
which is, ‘site-navigation’.
* The Chrome AMP Validator extension is flagging 2 x errors, that I’ve described,
and when I put the code snippets in that I mention the errors disappear.
* Surely that means that I’ve added the right code, as it’s now validated?
* My only problem is, I don’t actually know where to insert it in the filing system
of my child theme. What I’m thinking is, if it’s the AMP plugin that’s outputting
the code, is it even possible for me to append it?
* This is the code that is throwing two errors.
* `