Opened 11 years ago

Closed 11 years ago

#7199 closed task (fixed)

dijit.InlineEditBox "edit widget" API docs -- add focusNode property

Reported by: humbletim Owned by: Douglas Hays
Priority: low Milestone: 1.2
Component: Dijit Version: 1.1.1
Keywords: dijit._InlineEditor dijit.InlineEditBox focusNode API Cc:
Blocked By: Blocking:

Description

the mini-API documented in dijit.InlineEditBox? should also mention the requirement of a focusNode property (set to valid DOM node).

For reference here's trunk's current API doc:

  // An edit widget must support the following API to be used:
  // String getDisplayedValue() OR String getValue()
  // void setDisplayedValue(String) OR void setValue(String)
  // void focus()

... perhaps:

	//		/*DomNode*/ focusNode

NOTE: I'm assuming focusNode is always needed for the focus/blur automagic -- if that's incorrect, the style-copying code in dijit._InlineEditor should also be updated to check for existence of focusNode before copying styles over?

Change History (3)

comment:1 Changed 11 years ago by bill

Milestone: tbd1.2
Owner: set to Douglas Hays

Doug, you changed this recently; I forget if focusNode is still necessary or not.

comment:2 Changed 11 years ago by Douglas Hays

Status: newassigned
Type: defecttask

comment:3 Changed 11 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

(In [14509]) Fixes #7199. Added doc reference to focusNode. Cleaned up code refernces to focusNode.

Note: See TracTickets for help on using tickets.