From 9bb73608765fa413623689fde672c7b3b4fd01f9 Mon Sep 17 00:00:00 2001 From: Giovane Date: Tue, 15 Sep 2015 16:47:33 -0300 Subject: #442 Renames feedback box trigger files --- web-ui/app/js/feedback/feedback.js | 39 ---------------------- web-ui/app/js/feedback/feedback_trigger.js | 39 ++++++++++++++++++++++ web-ui/app/js/page/default.js | 2 +- web-ui/app/js/views/templates.js | 2 +- web-ui/app/templates/feedback/feedback.hbs | 8 ----- web-ui/app/templates/feedback/feedback_trigger.hbs | 8 +++++ web-ui/test/spec/feedback/feedback.spec.js | 38 --------------------- web-ui/test/spec/feedback/feedback_trigger.spec.js | 38 +++++++++++++++++++++ 8 files changed, 87 insertions(+), 87 deletions(-) delete mode 100644 web-ui/app/js/feedback/feedback.js create mode 100644 web-ui/app/js/feedback/feedback_trigger.js delete mode 100644 web-ui/app/templates/feedback/feedback.hbs create mode 100644 web-ui/app/templates/feedback/feedback_trigger.hbs delete mode 100644 web-ui/test/spec/feedback/feedback.spec.js create mode 100644 web-ui/test/spec/feedback/feedback_trigger.spec.js diff --git a/web-ui/app/js/feedback/feedback.js b/web-ui/app/js/feedback/feedback.js deleted file mode 100644 index e98c3e30..00000000 --- a/web-ui/app/js/feedback/feedback.js +++ /dev/null @@ -1,39 +0,0 @@ -/* - * Copyright (c) 2015 ThoughtWorks, Inc. - * - * Pixelated is free software: you can redistribute it and/or modify - * it under the terms of the GNU Affero General Public License as published by - * the Free Software Foundation, either version 3 of the License, or - * (at your option) any later version. - * - * Pixelated is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU Affero General Public License for more details. - * - * You should have received a copy of the GNU Affero General Public License - * along with Pixelated. If not, see . - */ - -define(['flight/lib/component', 'views/templates', 'page/events', 'features'], - function (defineComponent, templates, events, features) { - 'use strict'; - - return defineComponent(function () { - this.render = function () { - this.$node.html(templates.feedback.feedback()); - }; - - this.onClick = function() { - this.trigger(document, events.ui.feedback.open); - }; - - this.after('initialize', function () { - if (features.isEnabled('feedback')) { - this.render(); - this.on('click', this.onClick); - } - }); - - }); -}); diff --git a/web-ui/app/js/feedback/feedback_trigger.js b/web-ui/app/js/feedback/feedback_trigger.js new file mode 100644 index 00000000..e98c3e30 --- /dev/null +++ b/web-ui/app/js/feedback/feedback_trigger.js @@ -0,0 +1,39 @@ +/* + * Copyright (c) 2015 ThoughtWorks, Inc. + * + * Pixelated is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * Pixelated is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with Pixelated. If not, see . + */ + +define(['flight/lib/component', 'views/templates', 'page/events', 'features'], + function (defineComponent, templates, events, features) { + 'use strict'; + + return defineComponent(function () { + this.render = function () { + this.$node.html(templates.feedback.feedback()); + }; + + this.onClick = function() { + this.trigger(document, events.ui.feedback.open); + }; + + this.after('initialize', function () { + if (features.isEnabled('feedback')) { + this.render(); + this.on('click', this.onClick); + } + }); + + }); +}); diff --git a/web-ui/app/js/page/default.js b/web-ui/app/js/page/default.js index 08296c33..e060714d 100644 --- a/web-ui/app/js/page/default.js +++ b/web-ui/app/js/page/default.js @@ -43,7 +43,7 @@ define( 'flight/lib/logger', 'page/logout', 'page/logout_shortcut', - 'feedback/feedback', + 'feedback/feedback_trigger', 'feedback/compose_feedback', 'page/version', ], diff --git a/web-ui/app/js/views/templates.js b/web-ui/app/js/views/templates.js index d0fc9893..2a990c7f 100644 --- a/web-ui/app/js/views/templates.js +++ b/web-ui/app/js/views/templates.js @@ -61,7 +61,7 @@ define(['hbs/templates'], function (templates) { version: window.Pixelated['app/templates/page/version.hbs'] }, feedback: { - feedback: window.Pixelated['app/templates/feedback/feedback.hbs'], + feedback: window.Pixelated['app/templates/feedback/feedback_trigger.hbs'], compose_feedback: window.Pixelated['app/templates/feedback/compose_feedback.hbs'] } }; diff --git a/web-ui/app/templates/feedback/feedback.hbs b/web-ui/app/templates/feedback/feedback.hbs deleted file mode 100644 index 7f3f8ef1..00000000 --- a/web-ui/app/templates/feedback/feedback.hbs +++ /dev/null @@ -1,8 +0,0 @@ - diff --git a/web-ui/app/templates/feedback/feedback_trigger.hbs b/web-ui/app/templates/feedback/feedback_trigger.hbs new file mode 100644 index 00000000..7f3f8ef1 --- /dev/null +++ b/web-ui/app/templates/feedback/feedback_trigger.hbs @@ -0,0 +1,8 @@ + diff --git a/web-ui/test/spec/feedback/feedback.spec.js b/web-ui/test/spec/feedback/feedback.spec.js deleted file mode 100644 index f7011ea9..00000000 --- a/web-ui/test/spec/feedback/feedback.spec.js +++ /dev/null @@ -1,38 +0,0 @@ -describeComponent('feedback/feedback', function () { - 'use strict'; - - describe('Feedback link', function () { - var features; - - beforeEach(function() { - features = require('features'); - }); - - it('Should provide feedback link if logout is enabled', function () { - spyOn(features, 'isEnabled').and.returnValue(true); - this.setupComponent('', {}); - - var feedback_link = this.component.$node.find('a')[0]; - expect(feedback_link).toExist(); - }); - - it('Should not provide feedback link if disabled', function() { - spyOn(features, 'isEnabled').and.returnValue(false); - this.setupComponent('', {}); - - var feedback_link = this.component.$node.find('a')[0]; - expect(feedback_link).not.toExist(); - }); - - it('Should trigger ui:feedback:open event on click', function () { - - this.setupComponent('', {}); - var spy = spyOnEvent(document, Pixelated.events.ui.feedback.open); - - this.$node.find('a').click(); - expect(spy).toHaveBeenTriggeredOn(document); - }); - - }); -}); - diff --git a/web-ui/test/spec/feedback/feedback_trigger.spec.js b/web-ui/test/spec/feedback/feedback_trigger.spec.js new file mode 100644 index 00000000..66a31781 --- /dev/null +++ b/web-ui/test/spec/feedback/feedback_trigger.spec.js @@ -0,0 +1,38 @@ +describeComponent('feedback/feedback_trigger', function () { + 'use strict'; + + describe('Feedback link', function () { + var features; + + beforeEach(function() { + features = require('features'); + }); + + it('Should provide feedback link if logout is enabled', function () { + spyOn(features, 'isEnabled').and.returnValue(true); + this.setupComponent('', {}); + + var feedback_link = this.component.$node.find('a')[0]; + expect(feedback_link).toExist(); + }); + + it('Should not provide feedback link if disabled', function() { + spyOn(features, 'isEnabled').and.returnValue(false); + this.setupComponent('', {}); + + var feedback_link = this.component.$node.find('a')[0]; + expect(feedback_link).not.toExist(); + }); + + it('Should trigger ui:feedback:open event on click', function () { + + this.setupComponent('', {}); + var spy = spyOnEvent(document, Pixelated.events.ui.feedback.open); + + this.$node.find('a').click(); + expect(spy).toHaveBeenTriggeredOn(document); + }); + + }); +}); + -- cgit v1.2.3