-
Notifications
You must be signed in to change notification settings - Fork 35
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
Improve the website layout #3
Comments
jussi-kalliokoski, since it seems like currently you are converting your README.md to HTML, I would suggest you play around with markdown-styles (npm install -g markdown-styles) and the generate-md command. The github theme looks good. The only issue is it creates an assets/css directory and thus complicates your web structure, but if you really wanted to, you could inline the css. |
@cipherboy thanks, I'll take a look! |
@jussi-kalliokoski can't get the npm dependencies installing. Think it's something to do with trash and OSX. |
@rob-balfre can you paste the error you're getting so I can take a look? I'm developing on OS X myself and can't reproduce. :/ Looks awesome! I'd be happy to take even the WIP in already. :P Don't worry about the code blocks / syntax highlighting, I've got something coming for that!
The docs are actually compiled to a JSON format already: http://jussi-kalliokoski.github.io/trine/docs/v0.0.0/index.json - it's defined in https://github.com/jussi-kalliokoski/trine/blob/master/tools/DocumentationBuilder/index.js#L48 |
I made a PR with my doc's updates and included my npm debug. :) |
@rob-balfre ahh, I see the issue now! Please update to latest npm and the issue should go away. :) |
@jussi-kalliokoski worked :) |
Currently the website looks terrible. If there's any takers for making it look prettier, would be much appreciated.
The text was updated successfully, but these errors were encountered: