Skip to content

Commit

Permalink
fix: address review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
kaustavb12 committed Apr 29, 2024
1 parent 07c4273 commit 97b8e3f
Showing 1 changed file with 2 additions and 17 deletions.
19 changes: 2 additions & 17 deletions video_iframe/public/js/src/video_iframe.js
Original file line number Diff line number Diff line change
@@ -1,21 +1,6 @@
/* Javascript for VideoIframeXBlock. */
function VideoIframeXBlock(runtime, element, data) {

function updateCount(result) {
$('.count', element).text(result.count);
}

var handlerUrl = runtime.handlerUrl(element, 'increment_count');

$('p', element).click(function(eventObject) {
$.ajax({
type: "POST",
url: handlerUrl,
data: JSON.stringify({"hello": "world"}),
success: updateCount
});
});

wrapper_width = $('.video_wrapper', element).width();
$('.video_wrapper', element).height(wrapper_width*0.6);

Expand All @@ -24,13 +9,13 @@ function VideoIframeXBlock(runtime, element, data) {
display_name = data['display_name']
iframe_link = data['iframe_link']

if(video_download_link || video_download_link!==""){
if(video_download_link){
$('.downloads_wrapper', element).css('display', 'flex');
$('.video_download_wrapper', element).show();
$('#video_download_link', element).attr('href', video_download_link)
}

if(captions_download_link || captions_download_link!==""){
if(captions_download_link){
$('.downloads_wrapper', element).css('display', 'flex');
$('.captions_download_wrapper', element).show();
$('#captions_download_link', element).attr('href', captions_download_link)
Expand Down

0 comments on commit 97b8e3f

Please sign in to comment.