Changeset 12501 for sites/trunk/browsehappy.com/public_html/functions.php

Timestamp:

03/27/2023 10:54:49 PM (3 years ago)

Author:

coffee2code

Message:

Browse Happy: Use localized domain for Chrome when locale is zh_CN.

Note: This has actually been in production since at least 2021.

This isn't an ideal solution given that someone preferring Chinese isn't necessarily in China and thus bound by their internet restrictions. Unfortunately, the site does not currently perform location detection at this time. But this change has yielded no complaints unlike the previous implementation.

Fixes ​https://github.com/WordPress/browsehappy/issues/40.

File:

• sites/trunk/browsehappy.com/public_html/functions.php (modified) (1 diff)

sites/trunk/browsehappy.com/public_html/functions.php

@@ -21 +21 @@
             'normalized' => 1, // just first number
             'facebook' => 'googlechrome',
-            'url' => 'https://www.google.com/chrome',
+            'url' => class_exists( 'Browse_Happy_Locale' ) && 'zh_CN' === Browse_Happy_Locale::locale() ? 'https://www.google.cn/chrome' : 'https://www.google.com/chrome',
             'info' => ( $latest_strings ?
                 __( '“Get more done with the new Google Chrome. A more simple, secure, and faster web browser than ever, with Google’s smarts built-in.”', 'browsehappy' )