Skip to content

Commit 8b458f4

Browse files
KuniwakKuniwak
committed
Merge pull request mixi-inc#17 from mixi-inc/improving
Fix issues found by review
2 parents 75a7917 + 1d67551 commit 8b458f4

File tree

11 files changed

+199
-195
lines changed

11 files changed

+199
-195
lines changed

‎README.md‎

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -536,11 +536,17 @@ CSS セレクタについては、[MDN のCSS リファレンス](https://develo
536536

537537
### ステージ2
538538

539-
DOM 要素のスタイルを変更するトレーニング
539+
DOM 要素の属性・テキストを変更する
540+
トレーニング
541+
542+
543+
544+
このステージでは、スタイルの変更や
545+
表示文字列を変更するやり方を学びます。
540546

541547

542548

543-
#### DOM 要素のスタイル
549+
#### DOM 要素の属性・テキスト
544550

545551
DOM 要素には、
546552

@@ -622,7 +628,7 @@ DOM の構造を変更するトレーニング
622628

623629
#### DOM の構造
624630

625-
このステージでは、DOM のスタイルではなく、
631+
このステージでは、DOM の属性ではなく、
626632
構造を変更するトレーニングをおこないます。
627633

628634
たとえば、書籍を検索する Web API を使って、

‎gulpfile.js‎

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
'use strict';
22

3-
var util = require('util');
43
var path = require('path');
54
var gulp = require('gulp-help')(require('gulp'));
65
var merge = require('merge-stream');
@@ -33,18 +32,6 @@ var tasks = [
3332
];
3433

3534

36-
tasks.forEach(function(task) {
37-
var run = require('gulp-run');
38-
var url = util.format('http://localhost:%d/%s/', serve.PORT, task.id);
39-
40-
gulp.task(task.id, task.help, ['lint-' + task.id], function() {
41-
// We expected that mocha-phantomjs print colorized results, but it isn't.
42-
// So, I take a fast way that is using gulp-run.
43-
return run('$(npm bin)/mocha-phantomjs ' + url + ' || true').exec();
44-
});
45-
});
46-
47-
4835
tasks.forEach(function(task) {
4936
var stage = path.join('public', task.id);
5037
var js = path.join(stage, '**/*.js');

‎package.json‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,8 @@
3131
"gulp-eslint": "^0.6.0",
3232
"gulp-help": "^1.3.3",
3333
"gulp-nodemon": "^1.0.5",
34-
"gulp-run": "^1.6.6",
3534
"gulp-util": "^3.0.4",
3635
"merge-stream": "^0.1.7",
37-
"mocha-phantomjs": "^3.5.3",
3836
"stream-combiner2": "^1.0.2"
3937
},
4038
"dependencies": {

‎public/index.html‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ <h1>mixi JavaScript Training</h1>
2626
</div>
2727
<ol>
2828
<li><a href="stage1">DOM 要素を取得するトレーニング</a></li>
29-
<li><a href="stage2">DOM 要素のスタイルを変更するトレーニング</a></li>
29+
<li><a href="stage2">DOM 要素の属性・テキストを変更するトレーニング</a></li>
3030
<li><a href="stage3">DOM の構造を変更するトレーニング</a></li>
3131
<li><a href="stage4">DOM イベントを利用するトレーニング</a></li>
32-
<li><a href="stage5">サーバーと通信するトレーニング</a></li>
32+
<li><a href="stage5">非同期処理を書くトレーニング</a></li>
3333
<li><a href="stage6">モジュールを実装するトレーニング</a></li>
34-
<li><a href="stage7">よくあるイディオムを読み書きするトレーニング</a></li>
34+
<li><a href="stage7">よくあるイディオムを読むトレーニング</a></li>
3535
</ol>
3636
</div>
3737
</body>

‎public/stage1/index.html‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@
4141
</script>
4242
<script>mocha.setup('bdd')</script>
4343
<script src="analytics.js"></script>
44-
<script>
45-
describe.skipWhenPhantomJS = window.mochaPhantomJS ? describe.skip : describe;
46-
</script>
4744
<script src="tests.js"></script>
4845
<script>
4946
if (window.mochaPhantomJS) { mochaPhantomJS.run(); }

0 commit comments

Comments
 (0)