The argument for it is better aesthetics. By looking at just the set of values on a given set of properties I can tell right away whether it is a literal value, mixin include, or calculated value.
Personally (so take this with a grain of salt) I don't buy the convention argument. I don't conceptualize Sass as if it was a program I'm writing: there are executable statements, sure, but they are all pretty much independent. As such, relying on convention from programming language at the expense of the flexibility of Sass seems a poor choice. For example, it seems like the you took inspiration from !important when you incorporated constants, but that of course clashes with the established convention of prefix ! being a negation operator. But of course, you may have had these discussion before, so I understand if you're not feeling it. Eric, I personally don't find your pastie matching with my style; It's an improvement, but it not good enough. I consider the parentheses as part of the argument list. But of course, I may be weird :) Oh well, such is the beauty of OSS; I'll just maintain code with my own personal flavors. You guys have done a great job with the code, it's very easy to understand and hack on. Kudos. Thanks, Scott On Oct 9, 4:56 pm, "Chris Eppstein" <[EMAIL PROTECTED]> wrote: > I actually agree, but I assumed it was what you wanted based on my reading > of the code. > For the use case Scott mentioned, I think he can do the following and be > almost as happy:http://pastie.textmate.org/private/n6ukjex5crqxoc3wtnyhq > > Of course, I continue to be a fan of making the parentheses optional ;-) > > Chris > > On Thu, Oct 9, 2008 at 4:16 PM, Nathan Weizenbaum <[EMAIL PROTECTED]> wrote: > > > I'm not sure we want to allow this. The convention for all programming > > languages and existing Sass code is not having whitespace between a > > function name and its arguments, so since Sass takes an opinionated > > stance on style, unless there's a convincing use case for it I'd rather > > it be illegal. > > > Chris Eppstein wrote: > > > It's a bug fix, space was always allowed by the parsing regex. Nathan, > > > I've applied this patch to the master branch, you can pull from me to > > > apply this change. > > > > chris > > > > On Wed, Oct 8, 2008 at 9:53 PM, Scott Fleckenstein > > > <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> wrote: > > > > Hi All, > > > > So, i've been really enjoying Sass, and one thing I find myself doing > > > is lining up the indentation of the property values, so you get > > things > > > like the following: > > > http://pastie.textmate.org/private/o6k301znsocxgxnta5ikjw > > > > I find that form very easy on the eyes, but unfortunately, putting > > > whitespace between the mixin name and the argument list throws an > > > exception. The patch is incredibly simple, and it would be great to > > > get to get it included. > > > > the patch: http://pastie.textmate.org/private/al1u0yf5hhahcbvqlz9uw > > > > Thanks, > > > Scott > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Haml" 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/haml?hl=en -~----------~----~----~----~------~----~------~--~---
