From: <th...@je...> - 2001-01-30 17:16:37
|
----- Original Message -----=20 From: Doug Melvin <do...@cr...> To: <dyn...@li...> Sent: Tuesday, January 30, 2001 19:50 Subject: Re: Re[2]: [Dynapi-Dev] [Bug #130357] small typo in list.js > I've just had that conversation with a co-worker. > Can anyone tell me why this is not proper? > The best my co-worker could come up with was 'it doesn't feel right' > .... >=20 >=20 > > Either way, we shouldn't be using if (this.selected=3Db). 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=3Db) { > > >> > > > >> > should be > > >> > > > >> > if (this.selected=3D=3Db) { > > >> > > > >> > 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=3Db > > > 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/ > > > > > > > > _______________________________________________ > > Dynapi-Dev mailing list > > Dyn...@li... > > http://lists.sourceforge.net/lists/listinfo/dynapi-dev >=20 >=20 >=20 >=20 >=20 |