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

Add ocamldoc generation and improve the Html.Create library #82

Merged

Conversation

chrismamo1
Copy link
Contributor

This is another attempt at merging my latest changes to the HTML and CSS libraries, as well as adding OCamlDoc to the Makefile.

The changes involve adding a table creation function to the Html.Create module, which aims to be more robust (albeit less efficient) than the current table generation method. In the long-term, I would like to expand OCaml COW's existing facilities for the generation/manipulation of web pages from within OCaml, as lately this is becoming my favourite way to build web pages. I am declaring this intent in order to become somewhat less of a rogue-contributor, randomly adding code here and there without any real plan or coordination with the COW team.

Another change which I would like to work towards integrating into the entire library sometime in the future is the addition of a color type to the Css module.

@avsm and @samoht, you have been wonderfully patient with my incompetence and I swear one day I am going to bake you guys cookies.

@samoht
Copy link
Member

samoht commented Aug 10, 2015

Thanks! Also you don't need to close and open a new pull-request every time you force push to your branch. GitHub associates pull-requests to branch, so if you update your branch it will automatically update the associated pull-request.

…he last commit. we obviously don't want to waste user's time with all that junk when they can just produce it on their own, so I removed it
@chrismamo1 chrismamo1 force-pushed the gosh_darned_rebasing_and_also_actual_work branch from 9554282 to 0e7f9e8 Compare August 10, 2015 17:10
@chrismamo1
Copy link
Contributor Author

Sorry, I'm aware of the automatic synchronization between this PR and my branch, but the old PR used my master branch, which @avsm informed me is typically considered to be poor form.

…sensible order. Also takes advantage of the potential benefits of partially applying this function.
@avsm avsm merged commit 51f3a04 into mirage:master Sep 24, 2015
@avsm
Copy link
Member

avsm commented Sep 24, 2015

thanks for all this work Chris! It'll be in a release shortly.

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

3 participants