InlineEditBox: design problem with throwing away srcNodeRef

[bill]

The 0.9 InlineEditBox? widget has an initial design flaw in the way that it displays editable, the view (read-only) version of the text.

Currently, it throws away srcNodeRef and replaces it with a <span> named "editable". It then tries to make the <span> look like the original srcNodeRef by copying styles, but that doesn't really work. For example, copying width will fail. If the original srcNodeRef is:

<div style="width: 50%">
...
</div>

Then it will end up setting the <span> to have width=600px, the computed width of the <div> at the time the widget was instantiated. But if the user resizes the browser window the <span> won't resize, even though it should go to 50%.

Also, copying the styles requires a call to getComputedStyle() during widget initialization which is against our rules, because it doesn't work on elements where one of the ancestors is display:none, at least not in safari. Which means there will be problems putting InlineEditBox? in the non-selected tab.

InlineEditBox? should work like 0.4 where we keep the original srcNodeRef in the document.

See related bug #4098.

[bill]

(In [10354]) Better testcase for how I want InlineEditBox? to work.

• shows how you should specify a width for inline elements similar to the way

you specify the width of an <input>... ie, a width that is unrelated to the width of the current value.

• Also gives much better test of inherited styles from <style> tags and from the

browser's builtin rules for <p>, <h3>, etc. (All such style info must appear in the non-editing version of the text and the edit version of the tet.)

• Also demonstrates a problem with specifying paragraphs of text, inserting

unwanted line breaks (see #4098 for possible solution).

Refs #4097, #4098, #4167.

[Douglas Hays]

InlineEditBox? being redesigned by bill

[bill]

(In [10631]) Start refactor of InlineEditBox? to keep original srcNode, and create edit widget on demand. Code not working (well) yet. Refs #4097, #4098.

[bill]

(In [10824]) Continue InlineEditBox? work. Since we can't compute the width of a <span> on IE, since it's display:inline, I had to put a width="..." parameter for the widget. Refs #4097, #4098.

[bill]

(In [10893]) First pass at getting save & cancel buttons working. Refs #4097, #4098.

[bill]

(In [10918]) Avoid screen jitter when switching from display mode to editor. Refs #4097, #4098.

[bill]

(In [10919]) Fix inline mode (ie, editors for numbers, etc. that are inline with a line of text). Refs #4097, #4098.

[bill]

(In [10920]) Make no-value indicator work, and let user specify arbitrary HTML to use when no value is specified. Fixes #4362. Also, due to this and all the previous checkins, fixes #4097, #4098.