Re[2]: [Dynapi-Dev] [Bug #130357] small typo in list.js

[Robert R. <rra...@ya...>]

Either way, we shouldn't be using if (this.selected=b).  That's not
very proper.

-- 
// Robert Rainwater

On 1/30/2001, 8:47:21 AM EST, Raides wrote about "[Dynapi-Dev] [Bug #130357] small typo in list.js":

> Michael Pemberton wrote:
>> 
>> please put the "typo" back.  What is does is set the value of this.selected to
>> b and then evaluates the returned value (the new value of this.selected).  It
>> is needed to change the selected state of the item.
>> 

>> > if (this.selected=b) {
>> >
>> > should be
>> >
>> > if (this.selected==b) {
>> >
>> > right?

>         I will answer both for NS6's sake:

>         This is indeed a bug in NS6 and prevents normal execution of the rest of the
> code. It should read:

> this.selected=b
> if(this.selected){
> ...
> }else{
> ...
> }

>         I have done this kind of changes all around my local dynapi distribution and
> those pesky errors that NS6 throwed at me and their side effect of code not
> executing in that function anymore disappeared. Other special perso-NS6-ality of
> this browser is that if you want to dynamically resize an image, you have to do
> it TWICE, using a setTimeout to delay appropiately (30 millisecs are enough) the
> effect. The code I use to test NS6 comes with this mail. It is spanish code and
> HTML, poorly documented but that can be executed in IE4.0 and above with no
> changes at all and some errors due to their different underlying model.

>         Raides J.


----------------------
DynAPI Snapshots: http://dynapi.sourceforge.net/snapshot/
DynAPI Homepage: http://dynapi.sourceforge.net/