browserclass 8.x-1.2

A bug-fix release. Most importantly, it fixes a JavaScript error on some iOS browsers that could stop other aggregated scripts on the page from running. Updating is recommended for all 8.x-1.x users.

Bug fixes

• Fixed TypeError: this.stristr(...).split is not a function on iOS browsers without "Version/" in the user agent (Firefox iOS, Google app). With JS aggregation enabled, this stopped other scripts in the same aggregate from running. #3595980
• Fixed a fatal error for chained tokens such as [site:browserclass:browser] and [user:browserclass:browser]. #3595982
• Fixed "Undefined array key" PHP warnings for some user agents (e.g. Chrome or Konqueror without a version). #3595982
• Fixed Opera Mini version detection and versions without a dot (e.g. Version/17). #3595983

Behaviour changes

• hook_browserclass_classes() is now invoked, as on Drupal 7. Before, it was never called on Drupal 8+. Classes from existing implementations now appear in the [browserclass:*] tokens. The body classes are built in JavaScript and are not affected. #3595982
• Opera Mini now gets operamini<version> (e.g. operamini9) instead of operaminimini/<version>, matching the tokens. #3595983
• Browsers whose version has no dot now get a versioned class (e.g. safari17) instead of a duplicate unversioned one. Unversioned classes such as safari and operamini are unchanged. #3595983

Other changes

• Added GitLab CI and fixed all reported coding standards, PHPStan, ESLint and spelling issues. #3595981 #3595983
• Rewrote js/browserclass.js. The minified file is transpiled to ES5, so it keeps working in older browsers. #3595983
• Added a README, browserclass.api.php and an npm build script for the minified JS. #3595982 #3595983

Contributors

Thanks to therobyouknow, balis_m, GeduR, tunic and JKingsnorth for reports, patches and reviews.