onclick assignment in loop not behaving as expected.

Welcome to TSDN!

Can you post the corresponding HTML?

offsetParent is not consistent across the browsers - see this link.

The HTML is as follows, it's the same for each segment:
[HTML]<div id="submitNews">
<p class="submissionInfo">Proin sed mi in enim consectetuer ullamcorper. Duis suscipit cursus turpis. Phasellus diam leo, dictum dapibus, semper ac, pharetra condimentum, leo.</p>
<fieldset class="newsForm">
<ul>
<li><label for="contact_name">Your Name:</label>
<input id="contact_name" type="text" value="<?=$contact_name;?>" size="20" maxlength="50" tabindex="1" /></li>
<li><label for="contact_email">Your E-Mail:</label>
<input id="contact_email" type="text" value="<?=$contact_email;?>" size="40" maxlength="100" tabindex="2" /></li>
<li><label for="contact_city">Your Website:</label>
<input id="contact_city" type="text" value="<?=$contact_city;?>" size="50" maxlength="100" tabindex="3" /></li>
<li><label for="contact_type">News Type:</label>
<select id="contact_type" name="contact_type" tabindex="4">
<option value="false">--- Select One ---</option>
<option value="companyGossip">Company Gossip</option>
<option value="event">Event</option>
<option value="productLaunch">Product Launch</option>
<option value="other">Other</option>
</select>
</li>
<li><label for="contact_message">What's New?:</label>
<textarea id="contact_message" rows="8" cols="40" tabindex="5"><?=$contact_message;?></textarea></li>
</ul>
</fieldset>
<fieldset class="submit">
<input class="submit" type="submit" name="contact_submit" value="Submit" tabindex="6" />
</fieldset>
<div><div class="sub_slidebar"></div></div> <!--Wrapper to fix crap IE bug-->
</div>[/HTML]
You'll notice that the sub_slider div it wrapped in another div. This is why i've had to use offsetParent, the additional div is needed to correct an IE processing issue when an absolute element is proceeds a floated element.

OffsetParent seems to be retreived correctly, at least in FF. But in FF the onclick portion still doesn't work.

Expand|Select|Wrap|Line Numbers

1.              slidebar[i].onclick = function(){
2.                 sub_expand(50, parentId, parentHeight);
3.             }            
4.  

Try this:

the real problem is, that we have to use a closure when assigning functions with a loop since 'i' would always be the last value of the loop ... so we have to build it the following way (besides the eval which is suggested in the previous post)

the following is what you did and i will always be the last value ... so we get the value 3 alerted:

to fix that issue we may use a 'explicit' closure here:

that will alert '0' as we want it ...

kind regards

@Logician = You method works perfectly for me, across all browsers I've tested so thanks you.

@gits = When I was reading your example I understood what you were saying especially seen as that's the behaviour my script was performing, but I could seem to translate your example into my script. :-( I new to javaScript so it's likely just me doing something wrong.

Thanks to both of you though. My issue is now resolved. :-)

hi ...

glad to hear that you got it working ... a small hint for your closure-adaption:

that should do the trick. and should work like the eval-like version ...

kind regards

I understand. Thanks.

So what are the benefits of using your method compared to the eval way?

hi ...

typically we never have to use the Function-constructor or the eval function in our javascript code ... but sometimes we cannot avoid it ... for example to eval a json-string that came as response of an ajax request. the main issue with that are performance issues ... have a look here ... and since we don't need to use it it seems to be 'inelegant' to do so :)

kind regards