Summary: | Web Inspector: show cursor location in DTE | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andrey Lushnikov <lushnikov> | ||||||||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Andrey Lushnikov <lushnikov> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, webkit.review.bot, yurys | ||||||||||||||
Priority: | P2 | ||||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||
Hardware: | All | ||||||||||||||||
OS: | All | ||||||||||||||||
Attachments: |
|
Description
Andrey Lushnikov
2013-01-31 06:48:48 PST
Created attachment 185773 [details]
Patch
Comment on attachment 185773 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=185773&action=review > Source/WebCore/inspector/front-end/SourceFrame.js:66 > + this._sourcePositionElement.addStyleClass("source-position"); .className = "source-frame-position" > Source/WebCore/inspector/front-end/SourceFrame.js:640 > + this._sourcePositionElement.textContent = "Line " + (textRange.endLine + 1) + ", Column " + (textRange.endColumn + 1); = WebInspector.UIString("Line %d, Column %d, line, column); > Source/WebCore/inspector/front-end/SourceFrame.js:647 > + text = (textRange.endLine - textRange.startLine + 1) + " lines, " + text; ditto Created attachment 185776 [details]
Patch
Created attachment 185777 [details]
selection-multiline
Created attachment 185778 [details]
selection of one line
Created attachment 185779 [details]
cursor coordinates
Comment on attachment 185776 [details]
Patch
Add them to localizedStrings.js
Created attachment 185787 [details]
Patch
Comment on attachment 185787 [details] Patch Clearing flags on attachment: 185787 Committed r141426: <http://trac.webkit.org/changeset/141426> All reviewed patches have been landed. Closing bug. |