Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Releases tab from header #6388

Closed
wants to merge 1 commit into from
Closed

Conversation

vjeux
Copy link
Contributor

@vjeux vjeux commented Mar 9, 2016

This pull request moves the content of the Releases tab to the version number next to the title. With the search bar, the header was getting too crowded.

  • I've cleaned up the search style a bit and made it look like the React one (with the background color).
  • I've also improved the styling of the versions page.

screen shot 2016-03-09 at 2 22 17 pm

This pull request moves the content of the Releases tab to the version number next to the title. With the search bar, the header was getting too crowded.

I've cleaned up the search style a bit and made it look like the React one (with the background color).
I've also improved the styling of the versions page
@@ -86,7 +86,7 @@ var Site = React.createClass({
apiKey: '2c98749b4a1e588efec53b2acec13025',
indexName: 'react-native-versions',
inputSelector: '#algolia-doc-search',
algoliaOptions: { facetFilters: [ "tags:${algoliaVersion}" ] }
algoliaOptions: { facetFilters: [ "tags:${algoliaVersion}" ], hitsPerPage: 5 }
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a temporary fix until they fix the bug on their end

@facebook-github-bot
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @skevy, @bestander and @vjeux 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 Mar 9, 2016
width: 240px;
}

@media screen and (max-width: 1085px) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed all the size specific versions, not very useful

@bestander
Copy link
Contributor

Love it!
If you are done with it then shipit :)

@vjeux
Copy link
Contributor Author

vjeux commented Mar 10, 2016

@facebook-github-bot shipit

@facebook-github-bot
Copy link
Contributor

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

@ghost ghost closed this in 8c2e531 Mar 10, 2016
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.

3 participants