Converting code from JS to Jquery

Question

I found a bit of code to run a tool bar and need a bit of help converting it to jquery so I can use my existing code with it.

scroll_final=document.body.scrollTop;
scroll_final=document.documentElement.scrollTop;


var toolbarid=document.getElementById('toolbar');
toolbarid.style.visibility='hidden';


toolbarid.style.opacity='1.00';
toolbarid.style.filter='alpha(opacity=100)';
toolbarid.style.visibility='visible';

As always...thank you for your help, Todd

Did you check the documentation?

SLaks
– SLaks

2012-02-09 02:50:11 +00:00
Commented Feb 9, 2012 at 2:50 — SLaks
– SLaks, Commented Feb 9, 2012 at 2:50
docs.jquery.com/Tutorials

user1106925
– user1106925

2012-02-09 02:52:53 +00:00
Commented Feb 9, 2012 at 2:52 — user1106925
– user1106925, Commented Feb 9, 2012 at 2:52

nnnnnn · Accepted Answer · 2012-02-09 03:01:31Z

1

If you look at the doco for the jQuery .scrollTop() method, at the bottom there are some comments about it working differently in different browsers if you're trying to get the top of the whole document. So depending on your browser try:

scroll_final = $("body").scrollTop();
scroll_final = $(document).scrollTop();
scroll_final = $(window).scrollTop();
scroll_final = $("html").scrollTop();

For your other code, something like this:

var $toolbar = $('#toolbar');
$toolbar.css("visibility", "hidden");

$toolbar.css({
    opacity : '1.00',
    filter : 'alpha(opacity=100)',
    visibility : 'visible'
});

I assume the part where you set it to hidden and the part where you set the other properties and make it visible are within different event handlers or something, because if that's all in the same block of code it will all run before the browser repaints.

Rather than setting the "visibility" property you can use:

$toolbar.hide();
// OR
$toolbar.fadeOut();

// and then
$toolbar.show();
// OR
$toolbar.fadeIn();

But note that .hide() is the equivalent of .css('display', 'none').

edited Feb 9, 2012 at 3:01

answered Feb 9, 2012 at 2:55

nnnnnn

150k30 gold badges210 silver badges247 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

JayC Over a year ago

Just adding a note: filter : 'alpha(opacity=100)' should be removed. jQuery knows how to translate opacity to IE8 and below filters whenever needed.

maddogandnoriko Over a year ago

Thank you very much...I knew all the correct methods, just couldn't connect them together.

Collectives™ on Stack Overflow

Converting code from JS to Jquery

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related