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.

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

report(i18n): localize CRC renderer strings #5730

Merged
merged 5 commits into from
Aug 2, 2018
Merged

Conversation

patrickhulce
Copy link
Collaborator

A lot of the text is actually just the path and origin of each request so there wasn't actually that much here.

@brendankenny
Copy link
Member

brendankenny commented Jul 30, 2018

Slight change to make translation easier.

before:
screen shot 2018-07-30 at 15 42 58

after:
screen shot 2018-07-30 at 15 46 15

(edit) more after:
image

@paulirish paulirish removed their request for review August 1, 2018 20:31
@paulirish paulirish added the i18n internationalization thangs label Aug 2, 2018
Copy link
Member

@brendankenny brendankenny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -405,6 +396,11 @@ Util.UIStrings = {
toplevelWarningsMessage: 'There were issues affecting this run of Lighthouse:',
/** Label preceding a pictorial explanation of the scoring scale: 0-50 is red (bad), 50-90 is orange (ok), 90-100 is green (good). These colors are used throughout the report to provide context for how good/bad a particular result is. */
scorescaleLabel: 'Score scale:',

/** String of text shown in a graphical representation of the flow of network requests for the web page. This label represents the initial network request that fetches an HTML page. This navigation may be redirected (eg. Initial navigation to http://example.com redirects to https://www.example.com). */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the redirected explanation may be overkill but w/e

@paulirish paulirish merged commit 29513d5 into master Aug 2, 2018
@paulirish paulirish deleted the i18n_crc_strings branch August 2, 2018 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n internationalization thangs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants