[Radiant] New metatags for keywords, description feature

[Sean Cribbs]

Jamey,

That's great.  In response to Jim, it was a common enough use case for 
people who do SEO for their sites and such a low overhead to implement, 
it seemed like a reasonable thing to add.  I'd appreciate debates on 
this matter, however, as I'm open to culling superfluous code from the core.

Sean

Jamey Cribbs wrote:
> I just implemented what Jim suggested in my project and it works
> great.  I created a meta_keywords and a meta_description page part in
> the pages that I want the tags.  The nice thing is that you can use
> the inherit="true" parameter.
>
>
>
> On Mon, May 19, 2008 at 11:52 AM, Jim Gay <jim at saturnflyer.com> wrote:
>   
>> I'm curious about why r:meta was implemented. Couldn't you get the same
>> result by using page parts and dropping them in the appropriate place in
>> your layout?
>> I haven't used r:meta yet, so perhaps I'm unaware of some particular benefit
>>
>> -Jim
>>
>> On May 19, 2008, at 11:47 AM, Sean Cribbs wrote:
>>
>>     
>>> Personally, I'd like to see inheritance of meta info be optional and not
>>> the default.
>>>
>>> Sean
>>>
>>> Chris Parrish wrote:
>>>       
>>>> I just wanted to mention that I just noticed this change and am glad to
>>>> see it implemented in core as well.  It eliminates the need for my page_meta
>>>> extension.
>>>>
>>>> I too, like the inheritance idea (an optional attribute of the tag) but I
>>>> would also like to see a feature I had implemented in my extension.
>>>>
>>>> I the 'tag' attribute (mine was 'as_tag'), I had allowed true | false |
>>>> unless_blank for the reverse case that Jamey is mentioning.  I have sites
>>>> where I *don't* want keywords to inherit.  They need to be explicitly
>>>> defined or else the page gets none.  So, by using the 'unless_blank' option,
>>>> I was able to prevent the page from kicking out an empty set of <meta> tags.
>>>>  I'd be happy to write up a patch for this if others think that this would
>>>> be helpful.
>>>>
>>>> -Chris
>>>>
>>>> Jamey Cribbs wrote:
>>>>         
>>>>> First of all, thanks very much to the dev team for all of the latest
>>>>> release activity.  It is greatly appreciated!
>>>>>
>>>>> I do have a question about the new metatags fields for keywords and
>>>>> description.  These work great, but I noticed that there is no way to
>>>>> set them to inherit (unless I am missing something).  It does not look
>>>>> like you can set these fields on a parent page and have the children
>>>>> inherit them.  I thought that this would be the desired behavior, but
>>>>> maybe I am missing something.
>>>>>
>>>>> Thanks.
>>>>>
>>>>> Jamey Cribbs
>>>>> _______________________________________________
>>>>> Radiant mailing list
>>>>> Post:   Radiant at radiantcms.org
>>>>> Search: http://radiantcms.org/mailing-list/search/
>>>>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>>>>
>>>>>           
>>>> _______________________________________________
>>>> Radiant mailing list
>>>> Post:   Radiant at radiantcms.org
>>>> Search: http://radiantcms.org/mailing-list/search/
>>>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>>>
>>>>         
>>> _______________________________________________
>>> Radiant mailing list
>>> Post:   Radiant at radiantcms.org
>>> Search: http://radiantcms.org/mailing-list/search/
>>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>>       
>> _______________________________________________
>> Radiant mailing list
>> Post:   Radiant at radiantcms.org
>> Search: http://radiantcms.org/mailing-list/search/
>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>
>>     
> _______________________________________________
> Radiant mailing list
> Post:   Radiant at radiantcms.org
> Search: http://radiantcms.org/mailing-list/search/
> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>
>