Skip to content

Commit 3fb9c7f

Browse files
terrillterrill
committed
Remove console logs
1 parent dbd0894 commit 3fb9c7f

6 files changed

Lines changed: 2 additions & 32 deletions

File tree

‎build/ableplayer.dist.js‎

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -196,11 +196,9 @@
196196
this.transcriptDivLocation = null;
197197
}
198198
if ($(media).data('include-transcript') !== undefined && $(media).data('include-transcript') === false) {
199-
200199
this.hideTranscriptButton = true;
201200
}
202201
else {
203-
204202
this.hideTranscriptButton = null;
205203
}
206204

@@ -221,10 +219,6 @@
221219
}
222220
}
223221

224-
225-
226-
227-
228222
// In "Lyrics Mode", line breaks in WebVTT caption files are supported in the transcript
229223
// If false (default), line breaks are are removed from transcripts in order to provide a more seamless reading experience
230224
// If true, line breaks are preserved, so content can be presented karaoke-style, or as lines in a poem
@@ -10292,11 +10286,7 @@ if (thisObj.useTtml && (trackSrc.endsWith('.xml') || trackText.startsWith('<?xml
1029210286
// previously set transcriptType to null since there are no <track> elements
1029310287
// check again to see if captions have been collected from other sources (e.g., YouTube)
1029410288

10295-
10296-
10297-
1029810289
if (this.captions.length && (!(this.usingYouTubeCaptions || this.usingVimeoCaptions))) {
10299-
1030010290
// captions are possible! Use the default type (popup)
1030110291
// if other types ('external' and 'manual') were desired, transcriptType would not be null here
1030210292
this.transcriptType = 'popup';

‎build/ableplayer.js‎

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -196,11 +196,9 @@
196196
this.transcriptDivLocation = null;
197197
}
198198
if ($(media).data('include-transcript') !== undefined && $(media).data('include-transcript') === false) {
199-
console.log('YOU WANT TO HIDE TRANSCRIPT BUTTON');
200199
this.hideTranscriptButton = true;
201200
}
202201
else {
203-
console.log('YOU DO NOT WANT TO HIDE TRANSCRIPT BUTTON');
204202
this.hideTranscriptButton = null;
205203
}
206204

@@ -220,10 +218,6 @@ console.log('YOU DO NOT WANT TO HIDE TRANSCRIPT BUTTON');
220218
this.transcriptType = 'popup';
221219
}
222220
}
223-
console.log('transcriptType: ' + this.transcriptType);
224-
console.log('transcriptDivLocation: ' + this.transcriptDivLocation);
225-
console.log('hideTranscriptButton: ' + this.hideTranscriptButton);
226-
227221

228222
// In "Lyrics Mode", line breaks in WebVTT caption files are supported in the transcript
229223
// If false (default), line breaks are are removed from transcripts in order to provide a more seamless reading experience
@@ -10291,12 +10285,8 @@ if (thisObj.useTtml && (trackSrc.endsWith('.xml') || trackText.startsWith('<?xml
1029110285
if (!this.transcriptType) {
1029210286
// previously set transcriptType to null since there are no <track> elements
1029310287
// check again to see if captions have been collected from other sources (e.g., YouTube)
10294-
console.log("there is no transcriptType yet");
10295-
console.log('captions length: ' + this.captions.length);
10296-
console.log('usingYouTubeCaptions: ' + this.usingYouTubeCaptions);
1029710288

1029810289
if (this.captions.length && (!(this.usingYouTubeCaptions || this.usingVimeoCaptions))) {
10299-
console.log('captions are possible!');
1030010290
// captions are possible! Use the default type (popup)
1030110291
// if other types ('external' and 'manual') were desired, transcriptType would not be null here
1030210292
this.transcriptType = 'popup';

‎build/ableplayer.min.js‎

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "ableplayer",
3-
"version": "4.0.04",
3+
"version": "4.0.05",
44
"description": "fully accessible HTML5 media player",
55
"homepage": "http://ableplayer.github.io/ableplayer",
66
"bugs": "https://github.com/ableplayer/ableplayer/issues",

‎scripts/ableplayer-base.js‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -196,11 +196,9 @@
196196
this.transcriptDivLocation = null;
197197
}
198198
if ($(media).data('include-transcript') !== undefined && $(media).data('include-transcript') === false) {
199-
console.log('YOU WANT TO HIDE TRANSCRIPT BUTTON');
200199
this.hideTranscriptButton = true;
201200
}
202201
else {
203-
console.log('YOU DO NOT WANT TO HIDE TRANSCRIPT BUTTON');
204202
this.hideTranscriptButton = null;
205203
}
206204

@@ -220,10 +218,6 @@ console.log('YOU DO NOT WANT TO HIDE TRANSCRIPT BUTTON');
220218
this.transcriptType = 'popup';
221219
}
222220
}
223-
console.log('transcriptType: ' + this.transcriptType);
224-
console.log('transcriptDivLocation: ' + this.transcriptDivLocation);
225-
console.log('hideTranscriptButton: ' + this.hideTranscriptButton);
226-
227221

228222
// In "Lyrics Mode", line breaks in WebVTT caption files are supported in the transcript
229223
// If false (default), line breaks are are removed from transcripts in order to provide a more seamless reading experience

‎scripts/transcript.js‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,8 @@
88
if (!this.transcriptType) {
99
// previously set transcriptType to null since there are no <track> elements
1010
// check again to see if captions have been collected from other sources (e.g., YouTube)
11-
console.log("there is no transcriptType yet");
12-
console.log('captions length: ' + this.captions.length);
13-
console.log('usingYouTubeCaptions: ' + this.usingYouTubeCaptions);
1411

1512
if (this.captions.length && (!(this.usingYouTubeCaptions || this.usingVimeoCaptions))) {
16-
console.log('captions are possible!');
1713
// captions are possible! Use the default type (popup)
1814
// if other types ('external' and 'manual') were desired, transcriptType would not be null here
1915
this.transcriptType = 'popup';

0 commit comments

Comments
 (0)