hey, sorry for that, there was actually a javascript missing.. the patch is fine now.. thanks for the testing
On Fri, Sep 4, 2009 at 9:33 AM, Leo germani <[email protected]> wrote: > No. the patch is fine... I tested it here apllying on the top of the trunk > of wordpress repository. > > The Add button wont do anything only if you: > > . leave the name field blank > . use one of the protected names (Thumbnail, Medium, Large, Full size) > > Leo,, > > > On Fri, Sep 4, 2009 at 2:15 AM, John Blackbourn > <[email protected]<johnbillion%[email protected]> > > wrote: > >> This is a really good idea. +1 from me. >> >> Unfortunately your current patch isn't working for me. Clicking the >> 'Add' link does nothing. Have you missed out some Javscript in your >> patch? >> >> John. >> >> 2009/9/4 Leo germani <[email protected]>: >> > hi there, >> > >> > Ive just commited this patch and would be nice to have some feedback. >> > >> > http://core.trac.wordpress.org/ticket/8599 >> > >> > Apply the patch and go to Settings > Media. You will be able to set as >> many >> > sizes as you like. then upload some images and enjoy >> > >> > then >> > >> > cheers >> > >> > Leo,, >> > >> > -- >> > leogermani.pirex.com.br >> > leogermani.estudiolivre.org >> > _______________________________________________ >> > wp-testers mailing list >> > [email protected] >> > http://lists.automattic.com/mailman/listinfo/wp-testers >> > >> _______________________________________________ >> wp-testers mailing list >> [email protected] >> http://lists.automattic.com/mailman/listinfo/wp-testers >> > > > > -- > leogermani.pirex.com.br > leogermani.estudiolivre.org > -- leogermani.pirex.com.br leogermani.estudiolivre.org _______________________________________________ wp-testers mailing list [email protected] http://lists.automattic.com/mailman/listinfo/wp-testers
