diff options
author | Anton Timmermans <atimmer@git.wordpress.org> | 2018-06-07 15:07:31 +0000 |
---|---|---|
committer | Anton Timmermans <atimmer@git.wordpress.org> | 2018-06-07 15:07:31 +0000 |
commit | a14e4f1c869d6ebdeaa1618f0746b0c555fdd9ab (patch) | |
tree | a6999d44d39f78116250da440cfbe29502b371f3 | |
parent | 8a9bf2a08bc3f1727f1b5719f271763d1f1cfcc2 (diff) | |
download | wordpress-a14e4f1c869d6ebdeaa1618f0746b0c555fdd9ab.tar.gz wordpress-a14e4f1c869d6ebdeaa1618f0746b0c555fdd9ab.zip |
Docs: Improve JSDoc for `language-chooser.js`.
Props ireneyoast, manuelaugustin.
Fixes #43950.
git-svn-id: https://develop.svn.wordpress.org/trunk@43334 602fd350-edb4-49c9-b593-d223f7449a82
-rw-r--r-- | src/js/_enqueues/lib/language-chooser.js | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/src/js/_enqueues/lib/language-chooser.js b/src/js/_enqueues/lib/language-chooser.js index 628b8a81fb..9ca3ea679a 100644 --- a/src/js/_enqueues/lib/language-chooser.js +++ b/src/js/_enqueues/lib/language-chooser.js @@ -1,5 +1,8 @@ jQuery( function($) { - +/* + * Set the correct translation to the continue button and show a spinner + * when downloading a language. + */ var select = $( '#language' ), submit = $( '#language-continue' ); @@ -8,6 +11,10 @@ if ( ! $( 'body' ).hasClass( 'language-chooser' ) ) { } select.focus().on( 'change', function() { + /* + * When a language is selected, set matching translation to continue button + * and attach the language attribute. + */ var option = select.children( 'option:selected' ); submit.attr({ value: option.data( 'continue' ), @@ -16,8 +23,7 @@ select.focus().on( 'change', function() { }); $( 'form' ).submit( function() { - // Don't show a spinner for English and installed languages, - // as there is nothing to download. + // Show spinner for languages that need to be downloaded. if ( ! select.children( 'option:selected' ).data( 'installed' ) ) { $( this ).find( '.step .spinner' ).css( 'visibility', 'visible' ); } |