Change location & styling of dragdrop popups. · ableplayer/ableplayer@a149d51 · GitHub
Skip to content

Commit a149d51

Browse files
joedolsonjoedolson
committed
Change location & styling of dragdrop popups.
1 parent 1f198db commit a149d51

10 files changed

Lines changed: 56 additions & 32 deletions

File tree

‎build/ableplayer.dist.js‎

Lines changed: 9 additions & 5 deletions

‎build/ableplayer.js‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12939,10 +12939,14 @@ if (typeof module !== "undefined" && module.exports) {
1293912939
$autoScrollLabel = $("<label>", {
1294012940
for: "autoscroll-transcript-checkbox-" + this.mediaId,
1294112941
}).text(this.tt.autoScroll);
12942-
this.$transcriptToolbar.append(
12943-
$autoScrollLabel,
12944-
this.$autoScrollTranscriptCheckbox
12945-
);
12942+
$autoScrollContainer = $( '<div>', {
12943+
'class': 'autoscroll-transcript'
12944+
});
12945+
$autoScrollContainer.append(
12946+
$autoScrollLabel,
12947+
this.$autoScrollTranscriptCheckbox
12948+
);
12949+
this.$transcriptToolbar.append( $autoScrollContainer );
1294612950

1294712951
// Add field for selecting a transcript language
1294812952
// Only necessary if there is more than one language
@@ -15225,7 +15229,7 @@ if (typeof module !== "undefined" && module.exports) {
1522515229
else {
1522615230
// first, be sure window is on top
1522715231
this.updateZIndex(which);
15228-
popupTop = $windowButton.position().top + $windowButton.outerHeight();
15232+
popupTop = $toolbar.outerHeight() - 1;
1522915233
$windowPopup.css('top', popupTop);
1523015234
$windowPopup.show(200,'',function() {
1523115235
$windowButton.attr('aria-expanded','true');

‎build/ableplayer.min.css‎

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

‎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.

‎build/separate-dompurify/ableplayer.dist.js‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11570,10 +11570,14 @@ if (typeof module !== "undefined" && module.exports) {
1157011570
$autoScrollLabel = $("<label>", {
1157111571
for: "autoscroll-transcript-checkbox-" + this.mediaId,
1157211572
}).text(this.tt.autoScroll);
11573-
this.$transcriptToolbar.append(
11574-
$autoScrollLabel,
11575-
this.$autoScrollTranscriptCheckbox
11576-
);
11573+
$autoScrollContainer = $( '<div>', {
11574+
'class': 'autoscroll-transcript'
11575+
});
11576+
$autoScrollContainer.append(
11577+
$autoScrollLabel,
11578+
this.$autoScrollTranscriptCheckbox
11579+
);
11580+
this.$transcriptToolbar.append( $autoScrollContainer );
1157711581

1157811582
// Add field for selecting a transcript language
1157911583
// Only necessary if there is more than one language
@@ -13856,7 +13860,7 @@ if (typeof module !== "undefined" && module.exports) {
1385613860
else {
1385713861
// first, be sure window is on top
1385813862
this.updateZIndex(which);
13859-
popupTop = $windowButton.position().top + $windowButton.outerHeight();
13863+
popupTop = $toolbar.outerHeight() - 1;
1386013864
$windowPopup.css('top', popupTop);
1386113865
$windowPopup.show(200,'',function() {
1386213866
$windowButton.attr('aria-expanded','true');

‎build/separate-dompurify/ableplayer.js‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11584,10 +11584,14 @@ if (typeof module !== "undefined" && module.exports) {
1158411584
$autoScrollLabel = $("<label>", {
1158511585
for: "autoscroll-transcript-checkbox-" + this.mediaId,
1158611586
}).text(this.tt.autoScroll);
11587-
this.$transcriptToolbar.append(
11588-
$autoScrollLabel,
11589-
this.$autoScrollTranscriptCheckbox
11590-
);
11587+
$autoScrollContainer = $( '<div>', {
11588+
'class': 'autoscroll-transcript'
11589+
});
11590+
$autoScrollContainer.append(
11591+
$autoScrollLabel,
11592+
this.$autoScrollTranscriptCheckbox
11593+
);
11594+
this.$transcriptToolbar.append( $autoScrollContainer );
1159111595

1159211596
// Add field for selecting a transcript language
1159311597
// Only necessary if there is more than one language
@@ -13870,7 +13874,7 @@ if (typeof module !== "undefined" && module.exports) {
1387013874
else {
1387113875
// first, be sure window is on top
1387213876
this.updateZIndex(which);
13873-
popupTop = $windowButton.position().top + $windowButton.outerHeight();
13877+
popupTop = $toolbar.outerHeight() - 1;
1387413878
$windowPopup.css('top', popupTop);
1387513879
$windowPopup.show(200,'',function() {
1387613880
$windowButton.attr('aria-expanded','true');

‎build/separate-dompurify/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.

‎scripts/dragdrop.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@
435435
else {
436436
// first, be sure window is on top
437437
this.updateZIndex(which);
438-
popupTop = $windowButton.position().top + $windowButton.outerHeight();
438+
popupTop = $toolbar.outerHeight() - 1;
439439
$windowPopup.css('top', popupTop);
440440
$windowPopup.show(200,'',function() {
441441
$windowButton.attr('aria-expanded','true');

‎scripts/transcript.js‎

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,14 @@
7070
$autoScrollLabel = $("<label>", {
7171
for: "autoscroll-transcript-checkbox-" + this.mediaId,
7272
}).text(this.tt.autoScroll);
73-
this.$transcriptToolbar.append(
74-
$autoScrollLabel,
75-
this.$autoScrollTranscriptCheckbox
76-
);
73+
$autoScrollContainer = $( '<div>', {
74+
'class': 'autoscroll-transcript'
75+
});
76+
$autoScrollContainer.append(
77+
$autoScrollLabel,
78+
this.$autoScrollTranscriptCheckbox
79+
);
80+
this.$transcriptToolbar.append( $autoScrollContainer );
7781

7882
// Add field for selecting a transcript language
7983
// Only necessary if there is more than one language

‎styles/ableplayer.css‎

Lines changed: 8 additions & 4 deletions

0 commit comments

Comments
 (0)