Closed Bug 824243 Opened 8 years ago Closed 8 years ago
Profiler is not localizable
The new Profiler tool that landed in bug 795268 has its UI hard-coded in English. The only thing that is localizable is the tab in the developer tools window and the item in Firefox's Developer Tools menu.
Made all UI (incl. Cleopatra) strings localizable. Not that for Cleopatra I updated only those strings that are used in our Profiler.
Attachment #707880 - Flags: review?(rcampbell)
s/Not that/Note that/
Comment on attachment 707880 [details] [diff] [review] Make Profiler UI localizable Review of attachment 707880 [details] [diff] [review]: ----------------------------------------------------------------- This looks fine, although it totally breaks cleopatra out from the original repo. We're a total fork now. I guess this means we can seriously consider making it themeable as bug 823026 suggests. ::: browser/devtools/profiler/profiler.xul @@ +12,5 @@ > > +<!DOCTYPE window [ > +<!ENTITY % profilerDTD SYSTEM "chrome://browser/locale/devtools/profiler.dtd"> > + %profilerDTD; > +]> ok! ::: browser/locales/en-US/chrome/browser/devtools/profiler.properties @@ +52,5 @@ > +profiler.startProfiling=Click here %S to start profiling > + > +# LOCALIZATION NOTE (profiler.stopProfiling) > +# This string is displayed around the button that stops the profiler. > +profiler.stopProfiling = Click here %S to stop profiling do these need string arguments? If so you should explain what they are.
Attachment #707880 - Flags: review?(rcampbell) → review+
Added additional comments explaining string arguments for startProfiling and stopProfiling strings.
Rob, my plan is to push strings.js (although modified) upstream. Then all we will need to modify is strings.js which makes it easier to pull changes from upstream.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 21
Can someone explain what "self" is expected to mean? I'm trying the profiler but I'm not sure about it. I would also say that the strings displayed between the moment you stop the profiler and the graph is displayed are still hardcoded(processing working request..., finished and a couple others before these).
"Self" is how much time was spent doing work directly in that function. As opposed to the total time which is how much time was spent in that function and in functions it called. As for progress strings I was thinking about removing them at all (or replacing with "Loading...").
Filed bug 837735 about removing strings.
Thanks Anton, I think that adding this as a localization comment inside the file would be greatly appreciated by other localizers. P.S. If you go for "loading...", don't forget to use the proper single unicode character ("loading…")
You need to log in before you can comment on or make changes to this bug.