Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 481142

Summary:	[Quickfix] Quick fix directive inserted in the "wrong place" for <script> code
Product:	[ECD] Orion	Reporter:	Steve Northover <steve_northover>
Component:	JS Tools	Assignee:	Curtis Windatt <curtis.windatt.public>
Status:	RESOLVED FIXED	QA Contact:
Severity:	minor
Priority:	P3
Version:	10.0
Target Milestone:	11.0
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Steve Northover

2015-10-30 14:48:28 EDT

1) get minesweeper-objects
2) open counter.html
3) use the quick fix to add the globals directive for "Counter"
4) BUG: The directive is not correctly indented

Comment 1 Curtis Windatt

2015-11-02 10:26:39 EST

This is a problem in the quickfix.  It happens in both HTML embedded scripts and a simple JS file.

Comment 2 Curtis Windatt

2015-11-02 12:12:26 EST

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=f4a39e237d6586275058a57cac626db299b9c5bd
Mike and I paired to fix this in master.  Also fixes the same issues for adding eslint-env directive and tests.