RESOLVED FIXED 136980
Web Inspector: unlocalized 'many' string in type profiler inline bubble 
https://bugs.webkit.org/show_bug.cgi?id=136980
Summary Web Inspector: unlocalized 'many' string in type profiler inline bubble
Brian Burg
Reported 2014-09-20 23:20:42 PDT
I think this is supposed to be localized, right? You can see this in action on tetris.js:284
Attachments
patch (1.67 KB, patch)
2014-09-21 00:41 PDT, Saam Barati 		burg: review+
DetailsFormatted DiffDiff
patch (2.48 KB, patch)
2014-09-21 19:34 PDT, Saam Barati 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2014-09-20 23:20:55 PDT
<rdar://problem/18405527>
Timothy Hatcher
Comment 2 2014-09-20 23:23:21 PDT
It is debatable if they should be localized. We don't localize things that match language keywords or names. (For example the padding, margin, etc. labels in the Style Metrics pane are not localized.)
Saam Barati
Comment 3 2014-09-21 00:16:38 PDT
I think Joe and I spoke about this in a previous bug or over IRC. I think it should be localized because many JavaScript programmers will be familiar with String, Null, etc, just by programming in JavaScript, even though they may not understand the English meaning of these words. But I don't think non-English speaking programmers will immediately understand "many".
Saam Barati
Comment 4 2014-09-21 00:19:13 PDT
I guess an argument against my previous point is: It might be weird from a user experience perspective to have all type tokens be in English except the "many" token.
Saam Barati
Comment 5 2014-09-21 00:41:45 PDT
Created attachment 238423 [details] patch
Brian Burg
Comment 6 2014-09-21 18:54:50 PDT
(In reply to comment #4) > I guess an argument against my previous point is: > > It might be weird from a user experience perspective to have > all type tokens be in English except the "many" token. My inclination would be to put it in parens: (many) or call it multiple. From reading Japanese + code, it's not going to be a big shocker. But this discussion is mostly hypothetical until we get some localizers in the loop. For now, I think this should land as-is so we can start getting feedback and get rid of LOCALIZED STRING NOT FOUND.
Saam Barati
Comment 7 2014-09-21 19:34:47 PDT
Created attachment 238444 [details] patch Including parenthesis in localizable string.
WebKit Commit Bot
Comment 8 2014-09-21 20:42:41 PDT
Comment on attachment 238444 [details] patch Clearing flags on attachment: 238444 Committed r173803: <http://trac.webkit.org/changeset/173803>
WebKit Commit Bot
Comment 9 2014-09-21 20:42:44 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.