In my code I'm trying to set that if overflow is set to 'visible', it will remove the scrollbars using unloadScrollbars, and when overflow is set to 'hidden' it will not hide the scrollbars. I'm using edge animate (which supports javascript and css), and every time I make an if statement it always executes the first option, whether or not the parameter is true. Please help!
//overrides stage overflow setting
function overflow(toggle) {
document.getElementById('Stage').style.overflow = toggle;
return toggle;
}
overflow('hidden');
if (toggle='visible') {
unloadScrollBars();
console.log("unloading scrollbars");
} else {
console.log("not unloading scrollbars");
}
In my code I'm trying to set that if overflow is set to 'visible', it will remove the scrollbars using unloadScrollbars, and when overflow is set to 'hidden' it will not hide the scrollbars. I'm using edge animate (which supports javascript and css), and every time I make an if statement it always executes the first option, whether or not the parameter is true. Please help!
//overrides stage overflow setting
function overflow(toggle) {
document.getElementById('Stage').style.overflow = toggle;
return toggle;
}
overflow('hidden');
if (toggle='visible') {
unloadScrollBars();
console.log("unloading scrollbars");
} else {
console.log("not unloading scrollbars");
}
- Any working fiddles to show? – Muhammed Basil Commented Mar 21, 2014 at 16:20
- 2 use toggle == 'visible'. but toggle is never set. – user3401335 Commented Mar 21, 2014 at 16:21
3 Answers
Reset to default 8= is an assignment…
if (toggle='visible') {
Is the same as:
toggle='visible';
if (toggle) {
… you want a parison: == or ===.
if (toggle='visible') {
should be
if (toggle=='visible') {
toggle='visible'
This is assignment. (toggle='vivible') is always true in javascript.
toggle == 'visible'
This performs a parison, with type conversion if needed.
toggle === 'visible'
performs a parison of variable type and variable content.