Opened 12 years ago
Closed 12 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)
Change History (6)
Changed 12 years ago by
Attachment: | InlineEditBox.patch added |
---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
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 12 years ago by
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 12 years ago by
I have same effect with using the editor object's on onFocus/onBlur event.
Please close this ticket.
comment:5 Changed 12 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Ah OK, good idea using onFocus() and onBlur().
Sorry,
I missed [CLA] mark on the title.