diff --git a/src/js/select2/defaults.js b/src/js/select2/defaults.js index 836a3818..179c63e1 100644 --- a/src/js/select2/defaults.js +++ b/src/js/select2/defaults.js @@ -365,6 +365,7 @@ define([ maximumSelectionLength: 0, minimumResultsForSearch: 0, selectOnClose: false, + scrollAfterSelect: true, sorter: function (data) { return data; }, diff --git a/src/js/select2/results.js b/src/js/select2/results.js index 12b24df3..b56f8d3b 100644 --- a/src/js/select2/results.js +++ b/src/js/select2/results.js @@ -276,7 +276,10 @@ define([ } self.setClasses(); - self.highlightFirstItem(); + + if (self.options.get('scrollAfterSelect')) { + self.highlightFirstItem(); + } }); container.on('unselect', function () { @@ -285,7 +288,10 @@ define([ } self.setClasses(); - self.highlightFirstItem(); + + if (self.options.get('scrollAfterSelect')) { + self.highlightFirstItem(); + } }); container.on('open', function () { diff --git a/tests/results/focusing-tests.js b/tests/results/focusing-tests.js index 135d0016..35aa1247 100644 --- a/tests/results/focusing-tests.js +++ b/tests/results/focusing-tests.js @@ -135,4 +135,107 @@ test('results:append does not trigger results:focus', function (assert) { ] } }); +}); + +test('scrollAfterSelect triggers results:focus by default', function (assert) { + assert.expect(3); + + var $ = require('jquery'); + + var $select = $(''); + var $parent = $('
'); + + var $container = $(''); + var container = new MockContainer(); + + $parent.appendTo($('#qunit-fixture')); + $select.appendTo($parent); + + var Utils = require('select2/utils'); + var Options = require('select2/options'); + + var Results = require('select2/results'); + + var options = new Options({}); + var results = new Results($select, options); + + // Fake the data adapter for the `setClasses` method + results.data = {}; + results.data.current = function (callback) { + callback([{ id: 'test' }]); + }; + + results.render(); + + results.bind(container, $container); + + // check that default for scrollAfterSelect is true + assert.equal(options.get('scrollAfterSelect'), true); + + results.append({ + results: [ + { + id: 'test', + text: 'Test' + } + ] + }); + + results.on('results:focus', function (params) { + assert.equal(params.data.id, 'test'); + assert.equal(params.data.text, 'Test'); + }); + + container.trigger('select', {}); +}); + +test('!scrollAfterSelect does not trigger results:focus', function (assert) { + assert.expect(1); + + var $ = require('jquery'); + + var $select = $(''); + var $parent = $('
'); + + var $container = $(''); + var container = new MockContainer(); + + $parent.appendTo($('#qunit-fixture')); + $select.appendTo($parent); + + var Utils = require('select2/utils'); + var Options = require('select2/options'); + + var Results = require('select2/results'); + + var options = new Options({ scrollAfterSelect: false }); + var results = new Results($select, options); + + // Fake the data adapter for the `setClasses` method + results.data = {}; + results.data.current = function (callback) { + callback([{ id: 'test' }]); + }; + + results.render(); + + results.bind(container, $container); + + // check that default for scrollAfterSelect is false + assert.equal(options.get('scrollAfterSelect'), false); + + results.append({ + results: [ + { + id: 'test', + text: 'Test' + } + ] + }); + + results.on('results:focus', function () { + assert.ok(false, 'The results:focus event was triggered'); + }); + + container.trigger('select', {}); }); \ No newline at end of file