Attachment 248059 Details for Bug 448103 – Fix

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

[patch] Fix

Bug 448103 Outline selects wrong ranges.patch (text/plain), 1.18 KB, created by Curtis Windatt

on 2014-10-21 12:45:38 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Curtis Windatt

Created: 2014-10-21 12:45:38 EDT

Size: 1.18 KB

patch

obsolete

>diff --git a/bundles/org.eclipse.orion.client.webtools/web/webtools/cssOutliner.js b/bundles/org.eclipse.orion.client.webtools/web/webtools/cssOutliner.js
>index 0549e22..2324070 100644
>--- a/bundles/org.eclipse.orion.client.webtools/web/webtools/cssOutliner.js
>+++ b/bundles/org.eclipse.orion.client.webtools/web/webtools/cssOutliner.js
>@@ -62,18 +62,16 @@
> 							if (sel.parts && sel.parts.length > 0){
> 								var part = sel.parts[0]; // We want to check instanceof SelectorPart, but it is not API
> 								if (line === null) { line = part.line; }
>-								if (col === null) { col = part.col; }
>+								if (col === null) { col = part.col-1; }
> 								if (length === null){
> 									length = part.text.length;
>-									if (length > 0){ length--; /*length range is inclusive*/}
> 								}
> 							}
> 							// If no valid parts found, just use the entire selector text
> 							if (line === null) { line = sel.line; }
>-							if (col === null) { col = sel.col; }
>+							if (col === null) { col = sel.col-1; }
> 							if (length === null) {
> 								length = sel.text.length;
>-								if (length > 0){ length--; /*length range is inclusive*/}
> 							}
> 							selectorText.push(sel.text);
> 						}

Actions: View | Diff

Attachments on bug 448103: 248059 | 248062