InlineEditBox: set editors width to match original (view-mode) width

[Craig Baker]

The _inlineEditor subclass needs the width argument to know how to set it's width, but the inlineEditBox class doesn't pay attention to the width argument, so it must be styled. It would be better if one of the two arguments weren't needed (either the _inlineEditor reads off of the style or the inlineEditBox sets its style off of the width argument).

new dijit.InlineEditBox({
   title: "Label",
   renderAsHtml: true, 
   style: 'width: 200px', 
   width: 200
},this.labelEdit);

[Craig Baker]

Putting this in the postMixinProperities function (around line 125) fixed it for me.

if(this.width){

dojo.style(this.displayNode, 'width', this.width + 'px');

}

[Douglas Hays]

This is related to [10824]. I think the preferred dijit way of sizing elements is thru CSS, but the width attribute is apparently needed for IE. Also I think dojo.position (that we didn't have back in 1.0) could have been used to properly compute the size of a SPAN on IE and thus have been able to size the editor widget properly, so maybe an explicit width won't be needed in the future.

[bill]

running out of time for 1.7

[Colin Snover]

1.8 is frozen. Move all enhancements to next release. If you need an exemption from the freeze for this ticket, contact me immediately.

[dylan]

Given that no one has shown interest in creating a patch in the past 5+ years, I'm closing this as patchwelcome.