Skip to content

[Docs] fix props anchor links#6208

Closed
skv-headless wants to merge 2 commits into
react:masterfrom
skv-headless:fix-props-links
Closed

[Docs] fix props anchor links#6208
skv-headless wants to merge 2 commits into
react:masterfrom
skv-headless:fix-props-links

Conversation

@skv-headless

Copy link
Copy Markdown
Contributor

All anchor links are broken because of https://github.com/facebook/react-native/blob/master/website/core/Site.js#L37 base tag it means that relative urls won't work.
closes #5450
proof
screen shot 2016-02-29 at 13 34 35

@facebook-github-bot

Copy link
Copy Markdown
Contributor

By analyzing the blame information on this pull request, we identified @vjeux, @fkling and @sahrens to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Feb 29, 2016
@vjeux

vjeux commented Feb 29, 2016

Copy link
Copy Markdown
Contributor

@facebook-github-bot shipit

@facebook-github-bot

Copy link
Copy Markdown
Contributor

Thanks for importing. If you are an FB employee go to Phabricator to review.

@skv-headless

Copy link
Copy Markdown
Contributor Author

@vjeux I'm so sorry. I wanted to update PR to fix others anchor links. Take a look on second commit please.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@skv-headless updated the pull request.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@skv-headless updated the pull request.

@vjeux

vjeux commented Feb 29, 2016

Copy link
Copy Markdown
Contributor

@facebook-github-bot shipit

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@skv-headless updated the pull request.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@skv-headless updated the pull request.

@ghost ghost closed this in fcd468d Feb 29, 2016
@skv-headless

Copy link
Copy Markdown
Contributor Author

@vjeux again I'm sorry. At last update I fixed anchors on support page and clean code a little bit. Could you please ship again?

@vjeux

vjeux commented Feb 29, 2016

Copy link
Copy Markdown
Contributor

Erg sorry, I was too fast :p I can't shipit again but can you open up a pull request with the changes and i'll make sure to merge them in

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] ShadowPropTypesIOS style link broken

3 participants