Opened 10 years ago

Closed 10 years ago

#9677 closed enhancement (wontfix)

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

Reported by: youngho Owned by:
Priority: high Milestone: tbd
Component: Dijit Version: 1.3.2
Keywords: InlineEditBox Cc:
Blocked By: Blocking:

Description

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.

Attachments (1)

InlineEditBox.patch (525 bytes) - added by youngho 10 years ago.

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by youngho

Attachment: InlineEditBox.patch added

comment:1 Changed 10 years ago by youngho

Sorry,

I missed [CLA] mark on the title.

comment:2 Changed 10 years ago by bill

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

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.

comment:3 Changed 10 years ago by 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

comment:4 Changed 10 years ago by youngho

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

Please close this ticket.

comment:5 Changed 10 years ago by bill

Resolution: wontfix
Status: newclosed

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

Note: See TracTickets for help on using tickets.