On Thu, 2008-11-20 at 08:46 -0600, Jorge Vargas wrote:
>
> On Thu, Nov 20, 2008 at 3:45 AM, Florent Aide <[EMAIL PROTECTED]> wrote:
> >
> > On Thu, Nov 20, 2008 at 9:50 AM, Iain Duncan <[EMAIL PROTECTED]> wrote:
> >
> >> I noticed that previously, lines have been broken to meet PEP standards.
> >> I propose that we do *not* break lines at 80 chars in Genshi templates
> >> because:
> >>
> >> - the file is IMHO first and foremost an XML doc and not python code
> > yep
> >
> >> - in an XML doc, lines should preserve XML structure over PEP
> > as long as it is valid xml, your xml editor should be able to handle it...
> >
> >> - it's enormously more convenient to the designers and front end coders
> >> to have xml structure visible at a glance even if lines are long whereas
> >> broken up lines are kindof a pain unless they are reaaaallly long
> >
> > We need to get a balance here... Going over 80 is ok, because as you
> > said we don't have to follow pep8 for XML documents. But having line >
> > 100/150 chars is also a pain...
> >
> >> Thoughts? Correct me if I'm wrong please!
> >
> > You're the one working here... I say it's your call. We trust you to
> > do sane things.
> >
> agreed with all the above, another thing that should be done is
> spacing, current quckstart templates have 2 space indentation, I
> suggest we more that to 4spaces.
I actually prefer 2 myself for templates just to control indentation
amounts better, but good to hear.
> As for the formatting I don't know about you but I like my <li><a> as
> a one liner.
I the case of manuals I think that is often fine, but for looping
constructs I'm partial to laying it out. Here's what that is looking
like in my scratch pad:
<ul id="menu-left" class="component list-menu vert">
<li py:for="title,url in menu_side">
<a href="$url">$title</a>
</li>
</ul>
(NB: the id is optional and not required by the framework so that it is
easy for everyone to pop in their own choice of semantic ids. The
default css behaviours come from tag, class, and inheritance only.
Thanks
Iain
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"TurboGears Trunk" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/turbogears-trunk?hl=en
-~----------~----~----~----~------~----~------~--~---
