Fix two inline edit kludges

RESOLVED INVALID

Status

()

--
trivial
RESOLVED INVALID
18 years ago
10 years ago

People

(Reporter: neil, Assigned: bugs)

Tracking

Trunk
x86
Windows 95
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

18 years ago
Inline edit fields are not easily skinnable:
ileattr="text" can only be set on a text node;
rootcontent must be set to the tag name to which the binding will be bound.
I would like to use ile when neither of these apply.

field.setAttribute("value", txt.getAttribute("value"));
field.setAttribute("ileattr", "field");
field.setAttribute("rootcontent", txt.getAttribute("rootcontent"));

should be:

field.value = this.label;
field.setAttribute("ileattr", "field");
field.setAttribute("rootcontent", this.localName);

Or am I missing something?

Comment 1

18 years ago
->hyatt
Assignee: trudelle → hyatt

Comment 2

18 years ago
--> ben
Assignee: hyatt → ben
INVALID as we're not using this widget anymore. 

A new inline edit widget will appear soon. 
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → INVALID

Updated

10 years ago
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.