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

Improved Markdown renderer #125

Merged
merged 1 commit into from
Jul 11, 2016

Conversation

billiegoose
Copy link
Collaborator

If you'd like to adopt this repo, please open a few PRs and

I'll happily hand it over. :)

Well... I'm not eager to be saddled with the responsibility of this great project, but that may be the price I pay, lol.

My first contribution - big impact, low effort! It's an enhancement to the Web UI only. I swapped out the markdown renderer for a better one. In fact, after a quick hack and a pull-request, I got the same Markdown renderer used by www.npmjs.com to work in the browser.

BEFORE:

markdown

AFTER:

marky-markdown

@billiegoose
Copy link
Collaborator Author

Oops, forgot the live demos:
Old renderer: http://registry.node-modules.io/_browse/#/local-npm
New renderer: http://www2.node-modules.io/_browse/#/local-npm

@coveralls
Copy link

coveralls commented Jul 10, 2016

Coverage Status

Changes Unknown when pulling 67f95b8 on wmhilton:feature/marky-markdown into * on nolanlawson:master*.

@nolanlawson
Copy link
Member

Cool, this is really awesome! Thanks for the PR. :)

@nolanlawson nolanlawson merged commit ac72dae into local-npm:master Jul 11, 2016
@billiegoose
Copy link
Collaborator Author

Thank you for being awesome! Updated http://registry.node-modules.io/_browse

@billiegoose billiegoose deleted the feature/marky-markdown branch August 21, 2016 21:43
@billiegoose
Copy link
Collaborator Author

billiegoose commented Aug 22, 2016

close #110
edit: Le sigh. I guess I should stop just typing things and hoping they work and actually RTFM on Github

@zeke
Copy link
Contributor

zeke commented Aug 23, 2016

Nice! I'm so happy to see that in these dark times, some piece of the npm website still has an open-source future. Thanks for doing this, @wmhilton!

@billiegoose
Copy link
Collaborator Author

@zeke Feel free to publicize node-modules.io - would love for it to become a hub for like-minded people interested in disrupting the npm monopoly.

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.

None yet

4 participants