> Great work getting this to work for core. This will really,
> really, really help those of us running edge Clojure.
Thanks. Glad to help.
> BUGS:
Yup, know about all these. The right margin thing can be fixed by
making your window wider :-).
> ENHANCEMENTS:
> * Each namespace has a list of public variables/fns. It feels a bit
> long for some of the larger namespaces. Maybe a combination of ul
> elements and a jquery/script.acul.us expander div could go a long way?
Yeah, clojure.core in particular has a lot of stuff in it relative to
any of the namespaces in contrib. I'll think about the right approach.
I've generally been avoiding scripting to make the whole doc package
easier to deal with in a variety of scenarios (not just web hosted,
but downloadable, etc.).
> * In the HTML you use the same value in the id attribute of many
> tags. (e.g. id="var-tag", id="var-link"). Should you be setting the
> class attribute instead?
These ids aren't for styling but rather are used by the enlive
templating that builds the pages from fragments. So it doesn't matter
that they are duplicated.
But they probably should be classes anyway to be a little cleaner.
>
> Despite these minor concerns, I am very excited about this. Thank
> you for doing this for us.
No problem.
Enjoy!
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---
