[Ur] Code Review
Adam Chlipala
adamc at csail.mit.edu
Mon May 27 09:11:47 EDT 2013
Looks pretty good to me! A few observations:
- Why did you decide to parametrize your main functor over both a mode
and a function from modes to CSS classes? The functor implementation
seems to use the function and mode only by applying the former to the
latter, so why not just pass in a CSS class?
- Personally, I'd opt for the identifier [navBarClass] over
[getNavBarClass], since the former feels more functional-ish.
- There seem to be many extra blank lines at the end of the .ur file.
On 05/23/2013 04:33 AM, David Snider wrote:
> I've started writing a library that wraps the 'gui' library to produce
> widgets styled with twitter bootstrap.
> I would appreciate if somebody could take a quick look at a simple
> example I've done and tell me if I am using the language correctly.
>
> Thanks
>
> https://github.com/sniderd/bootstrap_gui
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.impredicative.com/pipermail/ur/attachments/20130527/66b78739/attachment.html>
More information about the Ur
mailing list