Skip to content

Add "Jump to Repository" Link #1147

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

Merged
merged 4 commits into from
Apr 24, 2017
Merged

Add "Jump to Repository" Link #1147

merged 4 commits into from
Apr 24, 2017

Conversation

skipjack
Copy link
Collaborator

Resolves #1107.

This adds another link beside the "Edit..." portal on external (fetched and generated) documentation. For example, you will now see something like this on any loader or plugin page:

image

Also made some alignment fixes and removed the icon for now as it looked a bit off. I hope to revisit and clean up the iconography for this website at some point as it's a bit messy at the moment.

…on (e.g. plugins/loaders)

Break page links out to their own component to isolate the logic and styling a bit. Remove
icons as their alignment and weights are a bit off. This should likely be re-visited later on
as they would look better with some nice icons.

Resolves #1107
Copy link
Collaborator

@simon04 simon04 left a comment

Choose a reason for hiding this comment

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

Awesome, I like the functionality.

I really liked the edit icon though. What about dropping the text and keeping the icons instead (for the repository link, the github icon from the first navigation bar could be used)?

@skipjack
Copy link
Collaborator Author

@simon04 thanks for the review! I'll add the edit icon back in then and continue to play around with this in separate branches. I like it as well but my designer side is super frustrated by the alignment and font-weight discrepancies that I saw when trying to add an icon for the new link (for consistency). Again, something I'm fine with coming back to.

@skipjack skipjack merged commit d8c0c19 into master Apr 24, 2017
@skipjack skipjack deleted the page-links-addition branch April 24, 2017 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plugin/Loader doc pages should have clear, explicit links to their respective repos
2 participants