Bug 153030 - Web Inspector: fix some common misspellings in localized strings
Summary: Web Inspector: fix some common misspellings in localized strings
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: BJ Burg
URL:
Keywords: DoNotImportToRadar
Depends on:
Blocks:
 
Reported: 2016-01-12 12:32 PST by BJ Burg
Modified: 2016-01-12 12:53 PST (History)
7 users (show)

See Also:

Attachments
Proposed Fix (5.47 KB, patch)
2016-01-12 12:34 PST, BJ Burg 		joepeck: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2016-01-12 12:32:28 PST 
<rdar://problem/24153340>
Comment 1 BJ Burg 2016-01-12 12:34:16 PST 
Created attachment 268799 [details]
Proposed Fix
Comment 2 Joseph Pecoraro 2016-01-12 12:37:02 PST 
Comment on attachment 268799 [details]
Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=268799&action=review

r=me!

> Source/WebInspectorUI/UserInterface/Views/DatabaseTableContentView.js:112
> +        this._messageTextViewElement = WebInspector.createMessageTextView(WebInspector.UIString("An error occurred trying to\nread the â%sâ table.").format(this.representedObject.name), true);

Is the newline really needed in here too?
Comment 3 BJ Burg 2016-01-12 12:52:38 PST 
Committed r194920: <http://trac.webkit.org/changeset/194920>
Comment 4 BJ Burg 2016-01-12 12:53:46 PST 
(In reply to comment #2)
> Comment on attachment 268799 [details]
> Proposed Fix
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=268799&action=review
> 
> r=me!
> 
> > Source/WebInspectorUI/UserInterface/Views/DatabaseTableContentView.js:112
> > +        this._messageTextViewElement = WebInspector.createMessageTextView(WebInspector.UIString("An error occurred trying to\nread the â%sâ table.").format(this.representedObject.name), true);
> 
> Is the newline really needed in here too?

I removed it before landing.