Bug 416933 - Run the standalone editor on IE8
Summary: Run the standalone editor on IE8
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Editor (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.0 M2   Edit
Assignee: Silenio Quarti CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks: 416947
  Show dependency tree
 
Reported: 2013-09-10 10:27 EDT by Silenio Quarti CLA
Modified: 2013-09-20 15:41 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Silenio Quarti CLA 2013-09-10 10:27:28 EDT
See bug#410841 for more details.
Comment 1 Silenio Quarti CLA 2013-09-10 12:34:05 EDT
This commit adds a simple ECMAScript 5 shim to the standalone editor. Only the necessary bits to get it running on IE8.  It also fixes the hit testing problems caused by a bug in TextRange.getClientRects() which returned an empty array if the start offset is zero.  There are also changes to fix content assist and tooltips (i.e. addEventListener).

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=ee654049bb9ec9c3cbcbdc0d35df79c7d9dcdf7c
Comment 2 Angelo ZERR CLA 2013-09-13 03:05:22 EDT
Silenio,

There is a bug, when you open the completion and try to close it when you click with mouse outside the popup completion.

Line 17172 the code is  : 

------------------
onMouseDown: function(event) {
	if (event.event.target.parentElement !== self.parentNode) {
		self.contentAssist.deactivate();
	}
...
------------------

and it should be :

------------------
onMouseDown: function(event) {
	var target = event.event.target || event.event.srcElement;
	if (target.parentElement !== self.parentNode) {
		self.contentAssist.deactivate();
	}
...
------------------
Comment 4 Angelo ZERR CLA 2013-09-19 04:19:50 EDT
(In reply to Silenio Quarti from comment #3)
> Thanks!
> 
> Fixed
> 
> http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/
> ?id=1943f2d0b5b4e0889a88a7682b93be94af266fd9

Hi Silenio,

Since I201309162230, the completion doesn't work with IE8. (201309152230 works great). (no javascript errors).

Manu thank's for your help.
Comment 5 Silenio Quarti CLA 2013-09-19 12:40:00 EDT
(In reply to Angelo ZERR from comment #4)
> Since I201309162230, the completion doesn't work with IE8. (201309152230
> works great). (no javascript errors).
> 
> Manu thank's for your help.

We started calling Array.isArray in content assist and we did not have a shim for it. Fixed

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=f9de99d820b5749a6f347bbb896240ce262718af