-
Notifications
You must be signed in to change notification settings - Fork 9.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
core(i18n): add strings for user timing entries table #5806
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (with changes)
@@ -27,7 +27,7 @@ const UIStrings = { | |||
columnScriptParse: 'Script Parse', | |||
/** A message displayed in a Lighthouse audit result warning that Chrome extensions on the user's system substantially affected Lighthouse's measurements and instructs the user on how to run again without those extensions. */ | |||
chromeExtensionsWarning: 'Chrome extensions negatively affected this page\'s load' + | |||
' performance. Try auditing the page in incognito mode or from a clean Chrome profile.', | |||
' performance. Try auditing the page in incognito mode or from a profile without extensions.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bikeshed: "from a Chrome profile without extensions"?
{key: 'timingType', itemType: 'text', text: 'Type'}, | ||
{key: 'startTime', itemType: 'ms', granularity: 0.01, text: 'Start Time'}, | ||
{key: 'duration', itemType: 'ms', granularity: 0.01, text: 'Duration'}, | ||
{key: 'name', itemType: 'text', text: UIStrings.columnName}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
str_(UIStrings.whatever)
for all of these
also a driveby fix for b/112186129