Re: [Radiant] New metatags for keywords, description feature

2008-08-22 Thread Adam van den Hoven
Jim, Where I work, I designed our system such that we have an in-memory representation of all the nodes in our site which was (somewhat) independent of the index.jsp that defines the node. We hang a lot of metadata off these nodes, a lot of which is inferred. This lets us do things like h

Re: [Radiant] New metatags for keywords, description feature

2008-08-22 Thread Jim Gay
I'm resurrecting this thread so that I can express my personal hatred of the meta tags. I propose that they be removed, or handled in a different way (and yes I volunteer myself to write the code for it). I'll be doing some thinking about and how to better handle the many options for HTML,

Re: [Radiant] New metatags for keywords, description feature

2008-05-20 Thread Chris Parrish
If you'd rather keep this as an extension, Sean, I'd be happy to improve my PageMeta Extension (it needs some tests, at least). I'd also entertain additional features, recommendations to improve my code, or even donate it to the core team's repository -- whatever -- if it would help Radiant.

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Chris Parrish
I can't speak for the core team but I wrote my own extension to do the same thing for clarity and simplicity in the UI. Page parts are for the page content (body) and the the other fields are for meta-data (kind of like HTML's vs. ). It could be done without (for Radiant or HTML) but is nice

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Jim Gay
Given the debate about how to implement an "inherit" functionality for something that already exists with r:content, I'd say this is bloat. I think there is a benefit in the user interface though, to have a text field rather than a text area to manage a small amount of content so I could be

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread 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

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Jamey Cribbs
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 <[EMAIL PROTECTED]> w

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Sean Cribbs
Nothing complicated, just something like: This would inherit if the field is blank (empty or nil). These would print a blank meta description tag if blank: Sean Chris Parrish wrote: I agree. Would you implement something like: (gotta work on that attrib name...) Or do you want

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Chris Parrish
I agree. Would you implement something like: (gotta work on that attrib name...) Or do you want some way to turn on inheritance on a per page basis (some pages a blank description inherits, on others it doesn't)? -Chris Sean Cribbs wrote: Personally, I'd like to see inheritance of met

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Jim Gay
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:

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Sean Cribbs
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 inh

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Chris Parrish
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 ext

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Jamey Cribbs
Sounds good. I will start working on the patch. Jamey On Mon, May 19, 2008 at 11:26 AM, Sean Cribbs <[EMAIL PROTECTED]> wrote: > Jamey, > > No, they do not inherit by default, but I'd entertain a patch to add that. > > Sean > > Jamey Cribbs wrote: >> >> First of all, thanks very much to the dev

Re: [Radiant] New metatags for keywords, description feature

2008-05-19 Thread Sean Cribbs
Jamey, No, they do not inherit by default, but I'd entertain a patch to add that. Sean 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 des

[Radiant] New metatags for keywords, description feature

2008-05-19 Thread Jamey Cribbs
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 some