Should refresh() set forceUpdate to true?

[njx]

We have another bug in the inline editors in Brackets that seems to have been caused by a change in CMv4, where the editor doesn't initially display until it's resized. (You can see this in the test file for #2420 - the line widget doesn't display its content until resize.) I thought this should be fixable by calling refresh() on the inline editor to force a display update after creating it, but it doesn't seem like refresh() sets forceUpdate. If I make it do so, then I can get this to work.

Would it make sense to either have refresh() always set forceUpdate, or have something like a force parameter to refresh() that sets it? (I feel like older versions of refresh() had this parameter, but maybe I'm misremembering...)

[marijnh]

I agree. I removed it because in normal situations a the call to regChange would force an update anyway, but your case (where the height is auto and zero) broke that assumption. Done in attached patch.

[njx]

Looks good, thanks.