Hi Pelle, On May 29, 2007, at 7:30 PM, Pelle Braendgaard wrote:
> Hi, > I've added the following patch: > http://dev.rubyonrails.org/attachment/ticket/8514/ > use_content_tag_helper_in_scaffold.diff > > It cleans up the html generated by the scaffold generator. I think this is a great idea. Alas, if I understand correctly, it isn't being accepted by core. :-( Is that true? If so, I'd love to see this made into a plug-in, so we could work on it further, and at least make it available to people who want it. Have you considered that? -enp > It doesn't > change anything major about the structure of the generated code, just > mainly ads id's and classes to the tr's using content_tag_for and > humanizes some labels that weren't there before. > > I would like to further add a patch after consulting with you guys > first about further possibly controversial changes I'd like to add to > it. > > Firstly I would like to remove the following fields from the generated > new and edit forms: > > * created_at, updated_at > * any foreign key field following standard ar foreign id naming > > My main reason for this is during rails conf I discovered that more > and more people are using the scaffold resource for creating code. > Almost everyone I talked to had these (well at least the date fields) > as pet pieves. > > My suggestion on how to do this is to add a method editable? to the > GeneratedAttribute class. This could then be simply tested in the > generators erb files. > > Regards > Pelle > > -- > http://extraeagle.com - Solutions for the electronic Extra Legal world > https://wideword.net - Encrypted wysiwyg document sharing > http://stakeventures.com - Bootstrapping blog > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
