[json] hover over color box doesn't show color picker. Fixes microsoft#32298

aeschli · aeschli · commit c185e7d44eea · 2017-08-12T16:40:09.000+02:00
diff --git a/extensions/json/client/src/colorDecorators.ts b/extensions/json/client/src/colorDecorators.ts
@@ -130,8 +130,8 @@ export function activateColorDecorations(decoratorProvider: (uri: string) => The
 }
 
 const ColorFormat_HEX = {
-	opaque: '#{red:X}{green:X}{blue:X}',
-	transparent: '#{red:X}{green:X}{blue:X}{alpha:X}'
+	opaque: '"#{red:X}{green:X}{blue:X}"',
+	transparent: '"#{red:X}{green:X}{blue:X}{alpha:X}"'
 };
 
 export class ColorProvider implements DocumentColorProvider {
@@ -144,7 +144,7 @@ export class ColorProvider implements DocumentColorProvider {
 		for (let range of ranges) {
 			let color = parseColorFromRange(document, range);
 			if (color) {
-				let r = new Range(range.start.line, range.start.character + 1, range.end.line, range.end.character - 1);
+				let r = new Range(range.start.line, range.start.character, range.end.line, range.end.character);
 				result.push(new ColorRange(r, color, [ColorFormat_HEX]));
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -130,8 +130,8 @@ export function activateColorDecorations(decoratorProvider: (uri: string) => The`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`const ColorFormat_HEX = {`
`133`		`- opaque: '#{red:X}{green:X}{blue:X}',`
`134`		`- transparent: '#{red:X}{green:X}{blue:X}{alpha:X}'`
	`133`	`+ opaque: '"#{red:X}{green:X}{blue:X}"',`
	`134`	`+ transparent: '"#{red:X}{green:X}{blue:X}{alpha:X}"'`
`135`	`135`	`};`
`136`	`136`
`137`	`137`	`export class ColorProvider implements DocumentColorProvider {`
`@@ -144,7 +144,7 @@ export class ColorProvider implements DocumentColorProvider {`
`144`	`144`	`for (let range of ranges) {`
`145`	`145`	`let color = parseColorFromRange(document, range);`
`146`	`146`	`if (color) {`
`147`		`- let r = new Range(range.start.line, range.start.character + 1, range.end.line, range.end.character - 1);`
	`147`	`+ let r = new Range(range.start.line, range.start.character, range.end.line, range.end.character);`
`148`	`148`	`result.push(new ColorRange(r, color, [ColorFormat_HEX]));`
`149`	`149`	`}`
`150`	`150`	`}`