diff options
| author | Phil Hughes <me@iamphill.com> | 2017-10-04 14:44:43 +0100 |
|---|---|---|
| committer | Phil Hughes <me@iamphill.com> | 2017-10-10 10:14:23 +0100 |
| commit | 6c97107d37771522d1deec6007a791332270ba6f (patch) | |
| tree | 07d8cf2d1dcd22af5eb4d41ebd81dd53204d4832 | |
| parent | bbf7ea24a560e22c3b51326d2413a6f8d6f0a530 (diff) | |
| download | gitlab-ce-6c97107d37771522d1deec6007a791332270ba6f.tar.gz | |
fixed eslint
| -rw-r--r-- | app/assets/javascripts/flash.js | 16 | ||||
| -rw-r--r-- | spec/javascripts/flash_spec.js | 5 |
2 files changed, 10 insertions, 11 deletions
diff --git a/app/assets/javascripts/flash.js b/app/assets/javascripts/flash.js index d745483c93f..8ac2b96a22d 100644 --- a/app/assets/javascripts/flash.js +++ b/app/assets/javascripts/flash.js @@ -44,14 +44,14 @@ const createFlashEl = (message, type) => ` * along with ability to provide actionConfig which can be used to show * additional action or link on banner next to message * - * @param {String} message Flash message text - * @param {String} type Type of Flash, it can be `notice` or `alert` (default) - * @param {Object} parent Reference to parent element under which Flash needs to appear - * @param {Object} actonConfig Map of config to show action on banner - * @param {String} href URL to which action config should point to (default: '#') - * @param {String} title Title of action - * @param {Function} clickHandler Method to call when action is clicked on - * @param {Boolean} fadeTransition Boolean to determine whether to fade the alert out + * @param {String} message Flash message text + * @param {String} type Type of Flash, it can be `notice` or `alert` (default) + * @param {Object} parent Reference to parent element under which Flash needs to appear + * @param {Object} actonConfig Map of config to show action on banner + * @param {String} href URL to which action config should point to (default: '#') + * @param {String} title Title of action + * @param {Function} clickHandler Method to call when action is clicked on + * @param {Boolean} fadeTransition Boolean to determine whether to fade the alert out */ const createFlash = function createFlash( message, diff --git a/spec/javascripts/flash_spec.js b/spec/javascripts/flash_spec.js index 1feb31f5c4f..66cc76ee626 100644 --- a/spec/javascripts/flash_spec.js +++ b/spec/javascripts/flash_spec.js @@ -135,7 +135,6 @@ describe('Flash', () => { describe('createFlash', () => { describe('no flash-container', () => { it('does not add to the DOM', () => { - const el = flash('test'); const flashEl = flash('testing'); expect( @@ -185,7 +184,7 @@ describe('Flash', () => { expect( document.querySelector('.flash-text').className, - ).toBe('flash-text container-fluid container-limited') + ).toBe('flash-text container-fluid container-limited'); }); it('does not add container when outside of content-wrapper', () => { @@ -194,7 +193,7 @@ describe('Flash', () => { expect( document.querySelector('.flash-text').className, - ).toBe('flash-text') + ).toBe('flash-text'); }); it('removes element after clicking', () => { |