On Sep 23, 2009, at 11:33 AM, Jim Gay wrote:
I don't like the if/unless_blank tags when we could do this with if_content.

. . .

What would everyone think about adding this, but altering the default
behavior so that if a page part is blank, it is considered not there.
So <r:if_content part="my_part" allow_blank="false"> would be your use
case to override and <r:if_content part="my_part"> would be the
default where allow_blank is assumed to be true.

The original intent behind if_content was the semantic "does this part exist?" It's not meant to be something that checks for whitespace. if_blank seems much more intention revealing (with that goal in mind). I'm not sure that I see the need for both a new attribute on the if_content tag and a new if_blank tag, so I favor just the if_blank tag.

--
John Long
http://wiseheartdesign.com
http://recursivecreative.com
_______________________________________________
Radiant mailing list
Post:   [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

Reply via email to