Skip to content
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.

Sign up for GitHub

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

Jump to bottom

core(fcp-3g): remove unused i18n for LR compatibility #7103

Merged
merged 4 commits into from
Jan 29, 2019
Merged

core(fcp-3g): remove unused i18n for LR compatibility #7103

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
735d32d
Self contain fcp-3g i18n.
exterkamp Jan 29, 2019
36646c9
Removed i18n
exterkamp Jan 29, 2019
1357591
actually in ms
exterkamp Jan 29, 2019
67de0cf
non breaking space!
exterkamp Jan 29, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 4 additions & 8 deletions lighthouse-core/audits/metrics/first-contentful-paint-3g.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,22 +7,18 @@

const Audit = require('../audit.js');
const regular3G = require('../../config/constants.js').throttling.mobileRegluar3G;
const i18n = require('../../lib/i18n/i18n.js');
const FCP = require('./first-contentful-paint.js');
const ComputedFcp = require('../../computed/metrics/first-contentful-paint.js');

const i18nFilename = require.resolve('./first-contentful-paint.js');
const str_ = i18n.createMessageInstanceIdFn(i18nFilename, FCP.UIStrings);

class FirstContentfulPaint3G extends Audit {
/**
* @return {LH.Audit.Meta}
*/
static get meta() {
return {
id: 'first-contentful-paint-3g',
title: str_(FCP.UIStrings.title),
description: str_(FCP.UIStrings.description),
title: 'First Contentful Paint (3G)',
description: 'First Contentful Paint 3G marks the time at which the first text or image is ' +
`painted while on a 3G network. [Learn more](https://developers.google.com/web/tools/lighthouse/audits/first-contentful-paint).`,
scoreDisplayMode: Audit.SCORING_MODES.NUMERIC,
requiredArtifacts: ['traces', 'devtoolsLogs'],
};
Expand Down Expand Up @@ -60,7 +56,7 @@ class FirstContentfulPaint3G extends Audit {
context.options.scoreMedian
),
rawValue: metricResult.timing,
displayValue: str_(i18n.UIStrings.seconds, {timeInMs: metricResult.timing}),
displayValue: `${metricResult.timing}\xa0ms`,
};
}
}
Expand Down