Skip to content

Commit

Permalink
Rewrite JavaScript files according to linter specs
Browse files Browse the repository at this point in the history
  • Loading branch information
LeonTenko committed Apr 12, 2017
1 parent e32e753 commit 5f981eb
Show file tree
Hide file tree
Showing 4 changed files with 31 additions and 29 deletions.
31 changes: 17 additions & 14 deletions app/assets/javascripts/application.js
Original file line number Diff line number Diff line change
@@ -1,19 +1,22 @@
// This is a manifest file that'll be compiled into application.js, which will include all the files
// listed below.
// This is a manifest file that'll be compiled into application.js,
// which will include all the files listed below.
//
// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
// Any JavaScript/Coffee file within this directory, lib/assets/javascripts,
// vendor/assets/javascripts, or any plugin's vendor/assets/javascripts
// directory can be referenced here using a relative path.
//
// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
// compiled file. JavaScript code in this file should be added after the last require_* statement.
// It's not advisable to add code directly here, but if you do, it'll appear
// at the bottom of the compiled file. JavaScript code in this file should be
// added after the last require_* statement.
//
// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
// Read Sprockets README
// (https://github.com/rails/sprockets#sprockets-directives) for details
// about supported directives.
//
//= require jquery
//= require tether
//= require bootstrap-sprockets
//= require jquery_ujs
//= require lib/date_picker
//= require turbolinks
//= require_tree .
// = require jquery
// = require tether
// = require bootstrap-sprockets
// = require jquery_ujs
// = require lib/date_picker
// = require turbolinks
// = require_tree .
10 changes: 5 additions & 5 deletions app/assets/javascripts/bets.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
$(document).on('turbolinks:load', function() {
$(".table-selectable tr").on('click', function(event){
$("tr").removeClass("table-success");
if (event.target.type !== "radio"){
$(":radio", this).trigger('click')
$('.table-selectable tr').on('click', function(event) {
$('tr').removeClass('table-success');
if (event.target.type !== 'radio') {
$(':radio', this).trigger('click');
}
$(this).addClass("table-success");
$(this).addClass('table-success');
});
});
11 changes: 5 additions & 6 deletions app/assets/javascripts/cable.js
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
// Action Cable provides the framework to deal with WebSockets in Rails.
// You can generate new channels where WebSocket features live using the rails generate channel command.
// You can generate new channels where WebSocket features live using the rails
// generate channel command.
//
//= require action_cable
//= require_self
//= require_tree ./channels
// = require action_cable
// = require_self
// = require_tree ./channels

(function() {
this.App || (this.App = {});

App.cable = ActionCable.createConsumer();

}).call(this);
8 changes: 4 additions & 4 deletions app/assets/javascripts/lib/date_picker.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
$(document).on("turbolinks:load", function(){
$(".jquery-datepicker").datetimepicker({
minDate: "0",
format: "Y-m-d G:i:s P"
$(document).on('turbolinks:load', function() {
$('.jquery-datepicker').datetimepicker( {
minDate: '0',
format: 'Y-m-d G:i:s P',
});
});

0 comments on commit 5f981eb

Please sign in to comment.