amp.dev icon indicating copy to clipboard operation
amp.dev copied to clipboard

amp-nested-submenu docs show import script

Open sebastianbenz opened this issue 5 years ago • 2 comments

amp-nested-submenu does not need to be explicitly imported (which will result in validation error). We need to handle the case in the overview table generator when the extension script does not need to be imported. It probably also makes sense to mention this in the docs.

//cc @CrystalOnScript for thoughts

sebastianbenz avatar May 19 '20 11:05 sebastianbenz

Is it possible to add frontmatter to component docs for required-script: true/false and replace the area in the table with "No Script Required read below for further details" or something when it doesn't? This case is tricky, since it does require amp-sidebar. Maybe replace the required script with that?

CrystalOnScript avatar May 27 '20 17:05 CrystalOnScript

@CrystalOnScript is this still valid?

patrickkettner avatar Apr 12 '21 21:04 patrickkettner