Skip to content

Commit b2c7703

Browse files
committed
module scope and benefits like import
1 parent f97a04b commit b2c7703

File tree

2 files changed

+6
-27
lines changed

2 files changed

+6
-27
lines changed

assets/script.js

Lines changed: 5 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,18 @@
1+
import MediaPlayer from './mediaPlayer.js';
2+
13
const video = document.querySelector('video');
24
const button = document.querySelector('button');
35

6+
const player = new MediaPlayer({ el: video });
7+
48
/*
59
// Funciona, pero no es muy extensible
610
button.onclick = () => video.play();
711
// Por eso desarrollamos una clase (obj en js) MediaPlayer
8-
*/
9-
10-
function MediaPlayer(config) {
11-
this.media = config.el;
12-
}
13-
14-
MediaPlayer.prototype.play = function() {
15-
this.media.play();
16-
};
17-
18-
MediaPlayer.prototype.pause = function() {
19-
this.media.pause();
20-
};
21-
22-
/*
23-
MediaPlayer.prototype.togglePlay = function() {
24-
if (this.media.paused) {
25-
this.play();
26-
} else {
27-
this.pause();
28-
}
29-
};
3012
3113
button.onclick = () => player.togglePlay();
3214
*/
3315

34-
const player = new MediaPlayer({ el: video });
35-
36-
3716
button.onclick = () => {
38-
player.media.paused? player.play() : player.pause()
17+
player.media.paused? player.play() : player.pause()
3918
}

index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,6 @@ <h1>MediPlayer.js</h1>
2222
<button>Play/Pause</button>
2323
</main>
2424

25-
<script src="./assets/script.js"></script>
25+
<script type="module" src="./assets/script.js"></script>
2626
</body>
2727
</html>

0 commit comments

Comments
 (0)