Merge pull request #38 from AHKol/fix-58693

Fix 58693

src/services/htmlFormatter.ts

@@ -42,6 +42,18 @@ export function format(document: TextDocument, range: Range | undefined, options
 		}
 		range = Range.create(document.positionAt(startOffset), document.positionAt(endOffset));
 
+		//Do not modify if substring in inside an element
+		let firstHalf = value.substring(0, startOffset);
+		let secondHalf = value.substring(endOffset, value.length);
+		if(new RegExp(/.*[<][^>]*$/).test(firstHalf) && new RegExp(/^[^<]*[>].*/).test(secondHalf) ){ 
+			//return without modification
+			value = value.substring(startOffset, endOffset);
+			return [{
+				range: range,
+				newText: value
+			}];
+		}
+
 		includesEnd = endOffset === value.length;
 		value = value.substring(startOffset, endOffset);
 

src/test/formatter.test.ts

@@ -193,4 +193,17 @@ suite('JSON Formatter', () => {
 
 		format(content, expected);
 	});
-});
+
+	test('bug 58693', () => {
+		var content = [
+			'<a class="btn| btn-link|"></a>'
+		].join('\n');
+
+		var expected = [
+			'<a class="btn btn-link"></a>'
+		].join('\n');
+
+		format(content, expected);
+	});
+
+});