ampproject / ampproject/samples

Replace nav links in AMP article

Open
#106 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
HTML
Stars
452
Forks
187
PR merge metrics
No merged PRs in 30d

Description

My code cleverly replaces the menu on the Guardian's AMP pages with a menu where everything links to the Shadow Reader. Hooray! Unfortunately it does no such thing for the top nav links.

Do it!

![screen shot 2018-05-01 at 3 53 20 pm](https://user-images.githubusercontent.com/29419498/39490769-e66d3722-4d57-11e8-8a25-57e5f0278a9e.png)

Contributor guide

No contributing guide indexed for this repository

Research direction

No file, test, or entry point is named. Start by locating the AMP article sample and the existing menu replacement, then inspect how the top navigation links are handled. Done means the top navigation links on Guardian AMP pages point to the Shadow Reader as requested.

Written by the indexing model from the issue text.

Assessment

Tech stack
html
Domain
frontend, web-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.