Go to the FreeLists Home Page Home Signup Help Login
 



[dokuwiki] || [Date Prev] [10-2007 Date Index] [Date Next] || [Thread Prev] [10-2007 Thread Index] [Thread Next]

[dokuwiki] Re: [PATCH] refactored media form to use new form class

  • From: "TNHarris" <telliamed@xxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Sat, 06 Oct 2007 17:15:49 -0400
On Fri, 5 Oct 2007 21:27:18 +0200, "Michael Klier" <chi@xxxxxxxxxxx>
said:
> 
> Yes, how would you go about that? Maybe you could patch the form code
> and I modify my patches when it`s merged into the main repo and then
> send them to the list again?
> 

Patch attached. Andi please apply.

On Sat, 06 Oct 2007 01:40:31 +0100, "Chris Smith" <chris@xxxxxxxxxxxxx>
said:
> 
> I say implement the combining of class values into a single class
> attribute and let the plugin authors work out what they can and can not
> do.
> 

Well, I just wrote it to overwrite the default attributes, except where
its mandatory then I remove it from the array before passing to
buildAttributes. It's simpler than combining and as I said before, I can
think of a reasonable scenario where combining would be necessary.
-- tom
telliamed@xxxxxxxxxxx

-- 
http://www.fastmail.fm - And now for something completely different?





[ Home | Signup | Help | Login | Archives | Lists ]

All trademarks and copyrights within the FreeLists archives are owned by their respective owners.
Everything else ©2007 Avenir Technologies, LLC.