Sounds good. See my comments below.<br><br><div><span class="gmail_quote">On 6/19/06, <b class="gmail_sendername">Deb Lewis</b> <<a href="mailto:djlewis@acm.org">djlewis@acm.org</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Broke up the original top-level index.html into: doc/index.html,<br>doc/guide.html, doc/directives.html (plus installation.html and<br>configuration.html that got started last week).<br><br>Did a bit for formatting and content-tweaking along the way; organized
<br>directives ref's so that there are functional groupings presented as well as<br>alphabetiacal index (I often want have a purpose in mind when looking for<br>something, thus the fcnal pt of view)<br><br>Review/revise as desired; nuke the original top-level
index.html if you're<br>happy with the new stuff.<br><br>Still needs misc. polishing and updates for some of our recent changes, I<br>think.<br><br>~ Deb<br><br>-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
<br>consider config default change: believe the admin controller should default<br>OFF, or at least only on by default in development environment - not clear<br>this should be on by default in production, developer probably needs to
<br>consider access restriction issues in that env.</blockquote><div><br>Yes, we can make the default to be off, but lets have the default environments/develoment.rb turn it on since I want to have it available for users. That way if they have the latest configs it will be on for dev only. If they don't then it is off.
<br> </div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --<br>Couple comments on directives observed while going over all the doc entries:
<br><br>mv:gen_render is kind of a funky name - if this is really only about<br>partials, then call it ... mv:gen_partial and simplify the attribute value.<br>If it's got more general uses in mind, include descr/example (fragment of
<br>content vs. the specific rails notion of partial).</blockquote><div><br>Yes, the name came from the combination of generate and render since it is doing a generate of a partial and outputting a render :partial. I guess we can rename it to gen_partial.
<br> </div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">consider: might make sense for default semantics of mv:generate to be<br>mv:omit_tag="true", I think this is perhaps the normal usage, *unless*
<br>specifically overridden by mv:omit_tag="false" *or* mv:gen_replace attribute<br>is also specified.</blockquote><div><br><br>On this one I left the outer tags in by default since if you are designing in wysiwyg editor, you need to have the same number of divs in the desigining view as what is rendered if you want the same output. If for instance each div adds some sort of style (offset, linefeed, etc.). So by automatically removing them could cause the output to look different than it did in the editor. So on this point I would prefer people to add the omit_tag if they really want it.
</div></div><br>