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

Assign lang upon the creation of the testing element #7

Merged
merged 2 commits into from
Nov 21, 2012

Conversation

amire80
Copy link
Contributor

@amire80 amire80 commented Nov 19, 2012

This fixes issue 5. Firefox appears to have more fine-grained control on
font configuration, which makes it return surprisng values in non-English
locales. Before this change the default font in the test suite and in
the actual element would be different if the OS locale wasn't English.
Hence, it's better to initialize the lang before initializing
tests and webfonts.

This fixes issue 5. Firefox appears to have more fine-grained control on
font configuration, which makes it return surprisng values ini non-English
locales. Before this change the default font in the test suite and in
the actual element would be different if the OS locale wasn't English.
Hence, it's better to initialize the lang before initializing
tests and webfonts.
@@ -33,10 +34,10 @@
if ( defaultFonts !== "" ) {
expectedResetFontFamilyList =
fontFamilyList( defaultFonts ).concat( expectedResetFontFamilyList );
console.log( expectedResetFontFamilyList );
Copy link
Member

Choose a reason for hiding this comment

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

Debugging code?

santhoshtr added a commit that referenced this pull request Nov 21, 2012
Assign lang upon the creation of the testing element
@santhoshtr santhoshtr merged commit f9739c9 into wikimedia:master Nov 21, 2012
@amire80 amire80 deleted the issue_5 branch December 16, 2018 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants