[patch] [cla] Make InlineEditBox onShow/onHide work

[youngho]

Sometimes, There is a need for InlineEditBox? editor onShow/onHide state change.

For example, if an InlineEditBox? is located in a wipeIn/Out area and want to controll deactivate wipeOut event during editor showing and activate wipeOut event again when editor is hiding.

Also because InlineEditBox? is inherited from _Widget, reckless user will expect to work onShow/onHide even though there is a comment in _Wedget.

[youngho]

Sorry,

I missed [CLA] mark on the title.

[bill]

I think you mentioned this on dojo-interest too. I was saying there that you could connect to edit() and _showText() to monitor this, although I guess you want a more official solution.

It doesn't seem like onShow()/onHide() are appropriate names since the InlineEditBox isn't being showing/hidden, but rather the editor widget is.

[youngho]

Hello Bill,

when I wrote this ticket, I was thinking to mention dojo-interest discussion with a linking Nabble list. But I realized that it is my preference not official list right ? So I skip it.

Anyway thanks for the mention.

I agreed the onShow/onHide of the InlineEditBox? and the editor object should be distinguished. But the name onShow/onHide itself for the InlineEditBox? is acceptable I think.

Thanks,

Youngho

[youngho]

I have same effect with using the editor object's on onFocus/onBlur event.

Please close this ticket.

[bill]

Ah OK, good idea using onFocus() and onBlur().