Great work, Ernesto ! I just tried the samples application and the only problem that I've found is related to the editable grids. In both Editable Grid and Editable TreeGrid the 'ESC' keypress doesn't work. In Editable TreeGrid I can edit only folders but I can't edit leafs.
Other than this I find it better than the YUI version. The column resizing and reordering works better than before! I'll update Brix and try it there before 6.4.0. On Mon, Dec 3, 2012 at 10:24 PM, Ernesto Reinaldo Barreiro < reier...@gmail.com> wrote: > Martin, > > All grid functionality seems to be ported to jquery... I have pushed the > changes master. > > > https://github.com/wicketstuff/core/commit/85920962aa601e6114234caa9e7b2304c603ba49 > > It would be nice if someone with a real application using inMethod grid can > give it a try. > > Cheers, > > Ernesto > > On Sat, Dec 1, 2012 at 5:40 PM, Ernesto Reinaldo Barreiro < > reier...@gmail.com> wrote: > > > Martin, > > > > On Sat, Dec 1, 2012 at 5:20 PM, Martin Grigorov <mgrigo...@apache.org > >wrote: > > > >> Well done, Ernesto ! > >> > >> Since YUI implementation will be preserved for some time I think it is > OK > >> to commit directly in master branch. > >> The announcement for WicketStuff 6.4.0 will have a special note about > this > >> change with a description how to use the old YUI impl. > >> > >> Ok. I will add a useYui flag and some javadoc for it, > > > > > >> Another problem that I see is that "Select All" checkbox is not selected > >> after using it. > >> > >> > > Thanks for the feedback, I will try fix that and the other issues > tomorrow. > > > > > >> > >> On Sat, Dec 1, 2012 at 5:07 PM, Ernesto Reinaldo Barreiro < > >> reier...@gmail.com> wrote: > >> > >> > Martin. > >> > > >> > On Wed, Nov 28, 2012 at 9:25 AM, Martin Grigorov < > mgrigo...@apache.org > >> > >wrote: > >> > > >> > > On Wed, Nov 28, 2012 at 9:22 AM, Ernesto Reinaldo Barreiro < > >> > > reier...@gmail.com> wrote: > >> > > > >> > > > Hi, > >> > > > > >> > > > On Wed, Nov 28, 2012 at 9:14 AM, Martin Grigorov < > >> mgrigo...@apache.org > >> > > > >wrote: > >> > > > > >> > > > > Hi, > >> > > > > > >> > > > > InMethod Grid is more smarter. It supports column reordering, > >> > resizing, > >> > > > > better Ajax support. But it is no active maintainer at the > moment. > >> > > > > Different community members provide patches when they need fixes > >> but > >> > > > that's > >> > > > > all. > >> > > > > It is also based on Yahoo UI v.2. > >> > > > > > >> > > > > > >> > > > Maybe it is time to try to rewrite all column reordering, > re-sizing > >> > logic > >> > > > based on jquery (instead of YUI). Now that jquery comes "for free" > >> with > >> > > > wicket 6.x > >> > > > > >> > > > > >> > > I'm sure many users will be happy of such change. > >> > > Even upgrading to YUI 3 will be a big improvement. > >> > > The same is valid for wicket-datetime module. > >> > > > >> > > > >> > I have managed to port grid functionality to use jquery. See [1] > (just > >> > deployed a temporary version of the sample application). > >> > > >> > Issues. > >> > > >> > 1- sorting columns do not update grid properly. > >> > 2- Tree grid column reordering do work yet. > >> > > >> > I will try to fix those tomorrow. > >> > > >> > The only needed dependency is jquery itself. > >> > > >> > I will add a flag that allows to choose between jquery and YUI backed > >> grid > >> > (with default to jquery;-). Shall I commit those changes? Create a > >> branch? > >> > Or a patch? > >> > > >> > References, > >> > > >> > 1-http://antiliasoft.com/inmethod-grid-examples/ > >> > > >> > -- > >> > Regards - Ernesto Reinaldo Barreiro > >> > Antilia Soft > >> > http://antiliasoft.com/ <http://antiliasoft.com/antilia> > >> > > >> > >> > >> > >> -- > >> Martin Grigorov > >> jWeekend > >> Training, Consulting, Development > >> http://jWeekend.com <http://jweekend.com/> > >> > > > > > > > > -- > > Regards - Ernesto Reinaldo Barreiro > > Antilia Soft > > http://antiliasoft.com/ <http://antiliasoft.com/antilia> > > > > > -- Martin Grigorov jWeekend Training, Consulting, Development http://jWeekend.com <http://jweekend.com/>
