<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Looks pretty good to me! A few observations:<br>
<br>
- 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?<br>
- Personally, I'd opt for the identifier [navBarClass] over
[getNavBarClass], since the former feels more functional-ish.<br>
- There seem to be many extra blank lines at the end of the .ur file.<br>
<br>
On 05/23/2013 04:33 AM, David Snider wrote:
<blockquote cite="mid:519DD43E.2090508@davidsnider.net" type="cite">
<meta http-equiv="content-type"
content="text/html; charset=ISO-8859-1">
I've started writing a library that wraps the 'gui' library to produce
widgets styled with twitter bootstrap.<br>
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.<br>
<br>
<meta http-equiv="content-type"
content="text/html; charset=ISO-8859-1">
Thanks<br>
<br>
<a moz-do-not-send="true"
href="https://github.com/sniderd/bootstrap_gui">https://github.com/sniderd/bootstrap_gui</a><br>
</blockquote>
</body>
</html>