I'm trying to use when
and done
to make the second function run only after the first one is executed, but it's not working.
What I'm trying to achieve here is:
First I use $("#jobshome").load("jobs/newest-jobs .js-toprow");
to load a div from an external page (In the same domain, no cross domain here), then after it is fully loaded, the next script, which is a simple text slider, will run.
But now the external HTML isn't loading inside the "#jobshome" anymore.
The result that I get when I don't use this approach is that the whole external div will load inside a single slider, and that's not what I need.
If I use the initial part like this it works, but not as expected:
$(function(){$("#jobshome").load("jobs/newest-jobs .js-toprow");});
$(function(){
//rotation speed and timer
My code is as follows:
jQuery(document).ready(function($) {
$.when(function() {
$("#jobshome").load("jobs/newest-jobs .js-toprow");
}).done(function() {
//rotation speed and timer
var speed = 3000;
var run = setInterval(rotate, speed);
var slides = $('.js-toprow');
var container = $('#jobshome');
var elm = container.find(':first-child').prop("tagName");
var item_height = container.height();
var previous = 'prevabc'; //id of previous button
var next = 'nextabc'; //id of next button
slides.height(item_height); //set the slides to the correct pixel height
container.parent().height(item_height);
container.height(slides.length * item_height); //set the slides container to the correct total height
container.find(elm + ':first').before(container.find(elm + ':last'));
resetSlides();
//if user clicked on prev button
$('#buttonsabc a').click(function(e) {
//slide the item
if (container.is(':animated')) {
return false;
}
if (e.target.id == previous) {
container.stop().animate({
'top': 0
}, 1500, function() {
container.find(elm + ':first').before(container.find(elm + ':last'));
resetSlides();
});
}
if (e.target.id == next) {
container.stop().animate({
'top': item_height * -2
}, 1500, function() {
container.find(elm + ':last').after(container.find(elm + ':first'));
resetSlides();
});
}
//cancel the link behavior
return false;
});
//if mouse hover, pause the auto rotation, otherwise rotate it
container.parent().mouseenter(function() {
clearInterval(run);
}).mouseleave(function() {
run = setInterval(rotate, speed);
});
function resetSlides() {
//and adjust the container so current is in the frame
container.css({
'top': -1 * item_height
});
}
});
});
//a simple function to click next link
//a timer will call this function, and the rotation will begin
function rotate() {
jQuery('#nextabc').click();
}
#carouselabc {
position: relative;
width: 60%;
margin: 0 auto;
}
#slidesabc {
overflow: hidden;
position: relative;
width: 100%;
height: 250px;
}
#areadoslideabc {
list-style: none;
width: 100%;
height: 250px;
margin: 0;
padding: 0;
position: relative;
}
#slidesabcdef {
width: 100%;
height: 250px;
float: left;
text-align: center;
position: relative;
font-family: lato, sans-serif;
}
/* Styling for prev and next buttons */
.btn-barabc {
max-width: 346px;
margin: 0 auto;
display: block;
position: relative;
top: 40px;
width: 100%;
}
#buttonsabc {
padding: 0 0 5px 0;
float: right;
}
#buttonsabc a {
text-align: center;
display: block;
font-size: 50px;
float: left;
outline: 0;
margin: 0 60px;
color: #b14943;
text-decoration: none;
display: block;
padding: 9px;
width: 35px;
}
a#prevabc:hover,
a#next:hover {
color: #FFF;
text-shadow: .5px 0px #b14943;
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<div id="carouselabc">
<div class="btn-barabc">
<div id="buttonsabc">
<a id="prevabc" href="#">Previous</a>
<a id="nextabc" href="#">Next</a>
</div>
</div>
<div id="slidesabc">
<div id="jobshome"></div>
</div>
</div>
Script source: https://codepen.io/TyStelmach/pen/yygvNK
Why dont you use the callback of load()
function?
$("#jobshome").load("jobs/newest-jobs .js-toprow", () => {
//do stuff after loading the html
});
when
doesn't work because it expects a Promise
or Deferred
object and if its neither it treats the value passed as a resolved Deferred
(which basically means it doesn't wait, but executes then
/done
right away).