Ryan,

No, you shouldn't have to make any changes to your extensions.

Sean

Ryan Heneise wrote:
> I like it - looks clean.
>
> Will we need to make any changes in current plugins that use Shards?
>
>
>
> On Nov 8, 2007, at 2:27 PM, Sean Cribbs wrote:
>
>   
>> Shards developers,
>>
>> I'm about to make a major change to the Shards extension's API and I
>> want a little feedback from those who are using it.  This is not a
>> breaking change, per se, but a behavioral one.  Let me set it up for  
>> you.
>>
>> Right now, the views (especially new/edit) for the  
>> Admin::PageController
>> are broken into a bajillion little pieces so that we can have many
>> points of insertion into the template.  This is great for flexibility,
>> but results in a bunch of tiny files and the need to dig through  
>> them to
>> find the proper region when you want to insert something.  I'd like to
>> dramatically reduce the number of partials that ship with the  
>> extension.
>>
>> The solution to this, which I have working, is to move the partials  
>> back
>> into one (or a few) files and provide a way to specify defaults, but
>> still allow overriding of the partials in the file system.  Here's an
>> example of what I'm talking about.  Normally, when you render a  
>> region,
>> a line like this exists in the template:
>>
>> <%= render_region :my_region %>
>>
>>
>> Via the 'admin' object, you would populate this region with partial
>> names, which would be rendered from the file system in the order
>> specified.  If this region has default partials, as do the ones that
>> come with Shards, they would instead be specified inline, where
>> appropriate.  Let's say "my_region" has default partials of "profile"
>> and "sidebar"; this is how you would specify them (notice the  
>> removal of
>> the =):
>>
>> <% render_region :my_region do |my| %>
>>  <% my.profile do %>
>>    <div id="profile">...</div>
>>  <% end %>
>>  <% my.sidebar do %>
>>    <div id="sidebar">...</div>
>>  <% end %>
>> <% end %>
>>
>> This adds a bit of weight and noise to the templates, but  
>> significantly
>> cuts down the number of partials.  Regions that don't have any default
>> partials could still be rendered in the manner shown in the first
>> example (with the =).  Let me know how you feel about this change.
>>
>> Sean
>> _______________________________________________
>> Radiant mailing list
>> Post:   [email protected]
>> Search: http://radiantcms.org/mailing-list/search/
>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>
>>     
>
> _______________________________________________
> Radiant mailing list
> Post:   [email protected]
> Search: http://radiantcms.org/mailing-list/search/
> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>
>   

_______________________________________________
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