tabindex (bug?) | WordPress.org

Resolved rsyd
(@rsyd)

6 years, 11 months ago
hello,

on amp dashboard didnt show any error.

but in chrome extension validator https://chrome.google.com/webstore/detail/amp-validator/nmoffdblmcmgeicmolmhobpoocbbmknc?hl=en says that role with on=”tap:AMP.setState(balbalblaba)” must have tabindex=”0″.. in result amp not valid..

example error
The attribute 'tabindex' in tag 'span' is missing or incorrect, but required by attribute 'on'.

Thanks.
- This topic was modified 6 years, 11 months ago by rsyd.
- This topic was modified 6 years, 11 months ago by rsyd.

Viewing 4 replies - 1 through 4 (of 4 total)

Plugin Author Weston Ruter
(@westonruter)

6 years, 11 months ago

Please share the URL of the AMP page that has the problem. Without that we cannot advise on what the problem is.
Thread Starter rsyd
(@rsyd)

6 years, 11 months ago
here the links
goto
- This reply was modified 6 years, 11 months ago by rsyd.
- This reply was modified 6 years, 11 months ago by rsyd.
Plugin Author Weston Ruter
(@westonruter)

6 years, 11 months ago

So this is your own custom AMP code? Just add the tabindex=0 attribute then.

The sanitizer is not catching this scenario at the moment. See https://github.com/ampproject/amp-wp/issues/1507

But since this looks to be your own custom code, it’s something you can fix easily yourself.

Thread Starter rsyd
(@rsyd)

6 years, 11 months ago

Yes, i just add manually to fix my self,, just can be better if plugin says the error warning

thanks

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘tabindex (bug?)’ is closed to new replies.

4 replies
2 participants
Last reply from: rsyd
Last activity: 6 years, 11 months ago
Status: resolved