AutoGrow 24px too high on IE

[Dinu]

On IE 7,8 the autogrow computed size is 24px too high; I've tracked it down to
newHeight = doc.getBody().$.scrollHeight + 24;
I don't know what that adjustment is for; I'm assuming it's to compensate for an eventual horizontal scrollbar, in which case:

• 24 is tentative, the scrollbars are not expected to have a certain size, so scrollbar height should be computed by other means
  
• only add height if a hrz scrollbar is actually present
  
• if additional height is added, schedule another resize to shrink editor back if scrollbars go away as result of the first resize; although it's counterintuitive (the horizontal scrollbar going away on a vertical resize, it's been known to happen)
  
  

I'll try to look some more at this and figure a way for myself

[Sa'ar Zac Elias]

Confirmed on Quirks mode.

[Sa'ar Zac Elias]

Fixed with [5924].

[Dinu]

Reopen: not fixed
Test: IE8/IE8 Standards, IE8 Compat/IE7 Standards

Problem:
CKEDITOR.env.ie&&CKEDITOR.env.quirks?0:24
Should (probably?) be the other way around:
CKEDITOR.env.ie&&CKEDITOR.env.quirks?24:0

Side comment:
According to ​http://msdn.microsoft.com/en-us/library/ms533687(VS.85).aspx compatMode is deprecated, use documentMode if available for quirks mode detection. Haven't tested, just a thought.

[Dinu]

Side note 2: CKEDITOR.env.ie&& is redundant

[Garry Yao]

Test: IE8/IE8 Standards, IE8 Compat/IE7 Standards

It works for me for those modes, could you attach a screenshot/screencast?

[Dinu]

Done. Your written code does this: adds 0px if the browser IS in quirks mode and 24px if it's in standards mode. Shouldn't it be the other way around?

[Dinu]

Please review... one simple thing less to maintain.

[Dinu]

Bump. Again need to repatch release.

[Garry Yao]

Please avoid adding comments to fixed ticket, #7173 has been opened for all issues mentioned.