From da9a849b4205ef03506f95cc6691ddf1543adaa7 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Sat, 14 Mar 2015 00:48:18 +0200 Subject: [PATCH] Show the Common language section only if the list is long enough --- src/jquery.uls.lcd.js | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/src/jquery.uls.lcd.js b/src/jquery.uls.lcd.js index b52dcee..84e2f24 100644 --- a/src/jquery.uls.lcd.js +++ b/src/jquery.uls.lcd.js @@ -115,6 +115,8 @@ render: function () { var $section, + language, + languagesCount = 0, lcd = this, regions = [], regionNames = { @@ -129,7 +131,24 @@ PA: 'Pacific' }; - regions.push( this.buildQuicklist() ); + // IE8 does not support Object.keys + if ( Object.keys ) { + languagesCount = Object.keys( this.options.languages ).length; + } else { + for ( language in this.options.languages ) { + if ( Object.prototype.hasOwnProperty.call( + this.options.languages, + language + ) ) { + languagesCount++; + } + } + } + + // Show the Common languages section, unless the list is very short + if ( languagesCount > 12 ) { + regions.push( this.buildQuicklist() ); + } $.each( $.uls.data.regiongroups, function ( regionCode ) { lcd.regionLanguages[ regionCode ] = [];