Re: disabled AjaxEditableMultiLineLabel

2007-11-23 Thread Al Maw
Gerolf Seitz wrote: i found the "problem": what i mentioned before with onBeforeRender didn't make it into rc1. sorry for messing that up. but now there are two checks for enabled/disabled. i actually prefer the one with dis-/enabling the label in onBeforeRender, because this way the onClick att

Re: disabled AjaxEditableMultiLineLabel

2007-11-22 Thread Gerolf Seitz
i found the "problem": what i mentioned before with onBeforeRender didn't make it into rc1. sorry for messing that up. but now there are two checks for enabled/disabled. i actually prefer the one with dis-/enabling the label in onBeforeRender, because this way the onClick attribute doesn't even ge

Re: disabled AjaxEditableMultiLineLabel

2007-11-22 Thread Francis De Brabandere
This is the code (wicket 1.3 rc1) add(new AjaxEditableMultiLineLabel("Comment", new PropertyModel(experiment, "experimentDetailStat.Comment")){ /* @Override protected void onEdit(AjaxRequestTarget target) { if(isEnabled()){

Re: disabled AjaxEditableMultiLineLabel

2007-11-21 Thread Gerolf Seitz
Francis, i would really be interested in your code that doesn't seem to work, if you don't mind. as of before Al's fix, it already worked for me because: the label is dis-/enabled in AjaxEditableLabel#onBeforeRender: label.setEnabled(isEnableAllowed() && isEnabled()); and since the LabelAjaxBehav

Re: disabled AjaxEditableMultiLineLabel

2007-11-21 Thread Francis De Brabandere
ok, i'll check on next release On Nov 21, 2007 12:56 PM, Al Maw <[EMAIL PROTECTED]> wrote: > Francis De Brabandere wrote: > > doing setEnabled(false) on a AjaxEditableMultiLineLabel does not disable > the > > editing feature > > > > this can be fixed by overriding onEdit(): > > > > @O

Re: disabled AjaxEditableMultiLineLabel

2007-11-21 Thread Al Maw
Francis De Brabandere wrote: doing setEnabled(false) on a AjaxEditableMultiLineLabel does not disable the editing feature this can be fixed by overriding onEdit(): @Override protected void onEdit(AjaxRequestTarget target) { if(isEnabled()){

Re: disabled AjaxEditableMultiLineLabel

2007-11-21 Thread Francis De Brabandere
we are using 1.3rc1 yes, I don't think it is implemented On Nov 21, 2007 12:39 PM, Gerolf Seitz <[EMAIL PROTECTED]> wrote: > what wicket version do you use? this should already work in 1.3rc1 > > Gerolf > > On Nov 21, 2007 12:20 PM, Francis De Brabandere <[EMAIL PROTECTED] > > wrote: > > > doing

Re: disabled AjaxEditableMultiLineLabel

2007-11-21 Thread Gerolf Seitz
what wicket version do you use? this should already work in 1.3rc1 Gerolf On Nov 21, 2007 12:20 PM, Francis De Brabandere <[EMAIL PROTECTED] > wrote: > doing setEnabled(false) on a AjaxEditableMultiLineLabel does not disable > the > editing feature > > this can be fixed by overriding onEdit():