search

svenmeier / wicket-dnd

Wicket drag-and-drop
20 stars 14 forks source link

AjaxIndicatorAware? #5

Closed d2a-raudenaerde closed 10 years ago

d2a-raudenaerde commented 10 years ago

Hi,

DropTarget extends AbstractDefaultAjaxBehavior, so it would seem adding a component that implements IAjaxIndicatorAware to the target in the onDrop() would trigger a indicator, but it doesn't.

Could you add an AjaxIndicatorAppender to the examples?

RobAu commented 10 years ago

See #6

svenmeier commented 10 years ago

As you already found out, an ajax indicator is not supported currently. I'll take a look at your patch.

svenmeier commented 10 years ago

DropTarget now uses the whole ajax attributes, so an indicator is now supported too.

RobAu commented 10 years ago

Sven, that is great! I was too focused on the indicator thing, your solution makes more sense.

Any chance it will get to a 0.6.2-SNAPSHOT in the near future?

svenmeier commented 10 years ago

0.6.1-SNAPSHOT is up already. Before releasing 0.6.1 I wanted to take a look at #2 (ontouchstart) first.