Skip to content
This repository was archived by the owner on Sep 10, 2022. It is now read-only.

Commit a580978

Browse files
committed
Merge pull request #21 from addyosmani/babel-update
Moving to new babelify
2 parents b505274 + 94c47c8 commit a580978

File tree

4 files changed

+13
-13
lines changed

4 files changed

+13
-13
lines changed

gulp-tasks/scripts.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ var eslint = require('gulp-eslint');
2222
var path = require('path');
2323
var glob = require('glob');
2424
var browserify = require('browserify');
25-
var babelify = require('babelify');
2625
var gutil = require('gulp-util');
2726
var source = require('vinyl-source-stream');
2827
var uglify = require('gulp-uglify');
@@ -45,7 +44,7 @@ function compileES6Bundles(browserifyBundles, cb) {
4544
var browserifyBundle = browserify({
4645
entries: [bundle.srcPath]
4746
})
48-
.transform(babelify);
47+
.transform('babelify', {presets: ['es2015']});
4948

5049
return browserifyBundle.bundle()
5150
.on('log', gutil.log.bind(gutil, 'Browserify Log'))

package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
"node": ">=0.10.0"
88
},
99
"devDependencies": {
10-
"babelify": "^6.3.0",
10+
"babel-preset-es2015": "^6.0.15",
11+
"babelify": "^7.2.0",
1112
"browserify": "^11.2.0",
1213
"del": "^2.0.2",
1314
"express": "^4.13.3",

src/scripts/controller/ApplicationController.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,16 @@ export default class ApplicationController extends Controller {
2525
constructor() {
2626
super();
2727

28-
this.navDrawer = new NavDrawerView();
28+
var navDrawer = new NavDrawerView();
2929

30-
this.sideNavToggleButton = document.querySelector('.js-toggle-menu');
31-
this.sideNavToggleButton.addEventListener('click', () => {
32-
this.navDrawer.toggle();
30+
var sideNavToggleButton = document.querySelector('.js-toggle-menu');
31+
sideNavToggleButton.addEventListener('click', () => {
32+
navDrawer.toggle();
3333
});
3434

3535
// TODO: Find more elegant solution to this and handling anchors in the
3636
// web app for dynamically loaded content
37-
var anchorElements = this.navDrawer.sideNavContent.querySelectorAll('a');
37+
var anchorElements = navDrawer.sideNavContent.querySelectorAll('a');
3838
for (var i = 0; i < anchorElements.length; i++) {
3939
if (!anchorElements[i].href) {
4040
continue;
@@ -43,7 +43,7 @@ export default class ApplicationController extends Controller {
4343
anchorElements[i].addEventListener('click', (clickEvent) => {
4444
clickEvent.preventDefault();
4545

46-
this.navDrawer.close();
46+
navDrawer.close();
4747

4848
var router = RouterSingleton.getRouter();
4949
router.goToPath(clickEvent.target.href);

src/scripts/controller/StaticPageController.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,11 @@ export default class StaticPageController extends Controller {
2323
constructor() {
2424
super();
2525

26-
this.navDrawer = new NavDrawerView();
26+
var navDrawer = new NavDrawerView();
2727

28-
this.sideNavToggleButton = document.querySelector('.js-toggle-menu');
29-
this.sideNavToggleButton.addEventListener('click', () => {
30-
this.navDrawer.toggle();
28+
var sideNavToggleButton = document.querySelector('.js-toggle-menu');
29+
sideNavToggleButton.addEventListener('click', () => {
30+
navDrawer.toggle();
3131
});
3232
}
3333
}

0 commit comments

Comments
 (0)