Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-15 Thread Jim Gay
I'm eager to see these additions in the core, but rather than just bugging them with pull requests would anyone like to give it a +/- 1 for adding in for 0.6.8? The changes in my repo add: 1) inherit = true to r:if_content and r:unless_content to work just like the r:content tag. 2) allow

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-15 Thread Sean Cribbs
Jim, I'm working on the core this week so I'll pull in those changes. Sean Jim Gay wrote: I'm eager to see these additions in the core, but rather than just bugging them with pull requests would anyone like to give it a +/- 1 for adding in for 0.6.8? The changes in my repo add: 1) inherit

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-10 Thread Jim Gay
I ended up going with find=any | all because it just made more sense when applied to unless_content. My additions are here: http://github.com/saturnflyer/radiant/tree ___ Radiant mailing list Post: Radiant@radiantcms.org Search:

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Jim Gay
Sean, what do you think? I think either r:if_content part=this, that select=all #this being the default r:if_content part=this, that select=any Or I'm thinking that inclusive=true might be good since we've got mostly true/false for extra attributes on r:content r:content [part=part_name]

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Sean Cribbs
Or I'm thinking that inclusive=true might be good since we've got mostly true/false for extra attributes on r:content r:content [part=part_name] [inherit=true|false] [contextual=true|false] / r:if_content [part=part_name, other_part] [inherit=true|false] [inclusive=true|false]

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Jim Gay
On Jul 8, 2008, at 2:27 PM, Sean Cribbs wrote: Or I'm thinking that inclusive=true might be good since we've got mostly true/false for extra attributes on r:content r:content [part=part_name] [inherit=true|false] [contextual=true|false] / r:if_content [part=part_name, other_part]

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Tim Gossett
On Tue, Jul 8, 2008 at 2:27 PM, Sean Cribbs [EMAIL PROTECTED] wrote: Or I'm thinking that inclusive=true might be good since we've got mostly true/false for extra attributes on r:content r:content [part=part_name] [inherit=true|false] [contextual=true|false] / r:if_content

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Jim Gay
On Jul 8, 2008, at 2:43 PM, Tim Gossett wrote: On Tue, Jul 8, 2008 at 2:27 PM, Sean Cribbs [EMAIL PROTECTED] wrote: Or I'm thinking that inclusive=true might be good since we've got mostly true/false for extra attributes on r:content r:content [part=part_name] [inherit=true|false]

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Chris Parrish
Since the all / find / require_all / inclusive attribute is both required and boolean (yeah I know XOR's been mentioned but I'm not going there), why not try to include that condition in the rest of it somehow? r:if_content part=my part (notice that the name part is singular) r:if_content

Re: [Radiant] if_content multiple parts (was if_content with inherit)

2008-07-08 Thread Jim Gay
On Jul 8, 2008, at 8:16 PM, Chris Parrish wrote: Since the all / find / require_all / inclusive attribute is both required and boolean (yeah I know XOR's been mentioned but I'm not going there), why not try to include that condition in the rest of it somehow? r:if_content part=my part