HeadersToolbar incompatible to other components than Label?

2010-09-23 Thread Benedikt Schlegel 
I've implemented a CheckGroupColumn which is supposed to easily extend
the DataTable with multi-row actions.
But when i add it to my DataTable i get the following exception:
---
WicketMessage: Component label must be applied to a tag of type
'input', not 'span wicket:id=label' (line 0, column 0)
[markup = 
jar:file:/D:/work/base/LIB_POOL/wicket/wicket-extensions-1.4.9.jar!/org/apache/wicket/extensions/markup/html/repeater/data/table/HeadersToolbar.html
wicket:panel
tr class=headers
span wicket:id=headers
th wicket:id=headerspan 
wicket:id=label[header-label]/span/th
/span
/tr
/wicket:panel, index = 6, current = 'span wicket:id=label' (line
21, column 27)]
---

I do understand whats wrong here, but i have no clue how to work a
round that problem. Any ideas?

Heres the CheckGroupColumn.java:
public class CheckGroupColumnT extends AbstractColumnT {
private static final long serialVersionUID = 1L;

private CheckGroupT checkGroup;

public CheckGroupColumn(CheckGroupT checkGroup) {
super(new ModelString());
this.checkGroup = checkGroup;
}

@Override
public Component getHeader(String componentId) {
return new CheckGroupSelector(componentId, checkGroup);
}

@Override
public void populateItem(ItemICellPopulatorT cellItem, String
componentId, IModelT rowModel) {
cellItem.add(new CheckT(componentId, rowModel, checkGroup));
}
}

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org

Re: HeadersToolbar incompatible to other components than Label?

2010-09-23 Thread Benedikt Schlegel 
In short, i want to add a CheckGroupSelector instead of a Label.
So i guess i need a way to replace the markup of HeaderToolbar,
without rewriting the whole thing.

Suggestions?

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org

Re: HeadersToolbar incompatible to other components than Label?

2010-09-23 Thread Martin Grigorov 
Yes, you'll need to provide your own markup.

MyToolbar extends HeaderToolbar {
// override the constroctor only
}
then provide MyToolbar.html which is modified version of HeaderToolbar.html

On Thu, Sep 23, 2010 at 4:46 PM, Benedikt Schlegel 
codecab.dri...@googlemail.com wrote:

 In short, i want to add a CheckGroupSelector instead of a Label.
 So i guess i need a way to replace the markup of HeaderToolbar,
 without rewriting the whole thing.

 Suggestions?

 -
 To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
 For additional commands, e-mail: users-h...@wicket.apache.org

Re: HeadersToolbar incompatible to other components than Label?

2010-09-23 Thread Benedikt Schlegel 
Well... most of the logic is in the constructor. So i have to copy the
whole code from there and just make some little tweaks.

Kinda ugly but now i realized that theres no other way. What a pity!

Thanks anyway.


2010/9/23 Martin Grigorov mgrigo...@apache.org:
 Yes, you'll need to provide your own markup.

 MyToolbar extends HeaderToolbar {
 // override the constroctor only
 }
 then provide MyToolbar.html which is modified version of HeaderToolbar.html

 On Thu, Sep 23, 2010 at 4:46 PM, Benedikt Schlegel 
 codecab.dri...@googlemail.com wrote:

 In short, i want to add a CheckGroupSelector instead of a Label.
 So i guess i need a way to replace the markup of HeaderToolbar,
 without rewriting the whole thing.

 Suggestions?

 -
 To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
 For additional commands, e-mail: users-h...@wicket.apache.org




-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org

Re: HeadersToolbar incompatible to other components than Label?

2010-09-23 Thread Igor Vaynberg 
instead of adding the checkbox directly wrap it in a panel.

-igor

On Thu, Sep 23, 2010 at 7:46 AM, Benedikt Schlegel
codecab.dri...@googlemail.com wrote:
 In short, i want to add a CheckGroupSelector instead of a Label.
 So i guess i need a way to replace the markup of HeaderToolbar,
 without rewriting the whole thing.

 Suggestions?

 -
 To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
 For additional commands, e-mail: users-h...@wicket.apache.org



-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org

Re: HeadersToolbar incompatible to other components than Label?

2010-09-23 Thread Benedikt Schlegel 
Works fine.

Thanks a lot!

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org