107734 - pariscope.fr - DynAPI JS Scroll2 object doesn't work

Status: VERIFIED FIXED

(Tech Evangelism Graveyard :: French, defect, P2)

Description

[Nicolas Fritsch]

In the "Nouveautés de la semaine" section, the JS scrollbar doesn't work on
BSD4.4: no scrolling.

Here is the JS script that creates this scrollbar but doesn't work:

<script language="Javascript"><!--

	onload=init;

	function init() {
		myscroll.activate();
		initMouseEvents();
	}
	
if (is.ns) {
		X = getObjCoordX(document.anchors['topleft']);
		Y = getObjCoordY(document.anchors['topleft']);
	}
	else {
		X = getObjCoordX(document.all['topleft']);
		Y = getObjCoordY(document.all['topleft']);
	}
	X += 6;
	Y += 4;
//alert(X + " - " + Y);
	myscroll = new Scroll(X,Y,426,362);
	myscroll.useH = false;
	myscroll.imgSet('/pics/'+univers+'/scroll2/',16,16,19);

	myscroll.build();

	writeCSS(myscroll.css);

	rollOver(univers,1);

//--></script>

<script language="Javascript"><!--
	document.writeln(myscroll.divStart);
//--></script>


<!-- Some HTML -->

<script language="Javascript"><!--
	document.writeln(myscroll.divEnd);
//--></script>

Comment 1

[Olivier Cahagne]

This site uses JavaScript-based browser detection from DynApi but the, this site
only uses the Netscape4 part of it (is.ns intead of is.ns5 + is.ns) or IE so I
guess document.anchors is not supported by Mozilla (couldn't find bug reports on
this).
(As they assume ns5+ns4 == ns in their above script)

Is document.anchors supported by Mozilla ?

From http://www.pariscope.fr/dynapi/js/browsercheck.js :

function BrowserCheck() {
	var b = navigator.appName
	if (b=="Netscape") this.b = "ns"
	else if (b=="Microsoft Internet Explorer") this.b = "ie"
	else this.b = b
	this.v = parseInt(navigator.appVersion)
	this.ns = (this.b=="ns" && this.v>=4)
	this.ns4 = (this.b=="ns" && this.v==4)
	this.ns5 = (this.b=="ns" && this.v==5)
	this.ie = (this.b=="ie" && this.v>=4)
	this.ie4 = (navigator.userAgent.indexOf('MSIE 4')>0)
	this.ie5 = (navigator.userAgent.indexOf('MSIE 5')>0)
	if (this.ie5) this.v = 5
	this.min = (this.ns||this.ie)
}

-> All (Win2k also affected with build 2001103003)

Comment 2

[Olivier Cahagne]

ok, mozilla supports document.anchors:

http://developer.netscape.com/evangelism/docs/technotes/xref/document-object/

So, this seems to be a real bug, not a Tech Evangelism bug. Can anyone confirm ?

Comment 3

[Phil Schwartau]

Attachment: Reduced testcase

Comment 4

[Phil Schwartau]

Confirming bug with Mozilla trunk binaries 20011030xx WinNT, Linux, Mac 9.1.
If you load the reduced testcase in NN4.7 or IE4, you get a red button in 
the Scroll object. This allows you to scroll down the long list of XXXX's.

In Mozilla, the button does not appear.

Comment 5

[Phil Schwartau]

Attachment: Reduced testcase: source

Comment 6

[Phil Schwartau]

Have tried to use the JS Debugger to pin this down, but it's
a little beyond me. Here is what I found, though:

Two of the most important JS files here are:

         http://www.pariscope.fr/dynapi/js/scroll2.js
         http://www.pariscope.fr/dynapi/js/dynlayer.js


The former file makes the DynAPI "Scroll2 Object", and it depends
on the latter file, which makes the DynAPI "DynLayer Object"


ScrollActivate() at scroll2.js line 152 calls these functions, among others:

  DynLayer()               dynlayer.js line 11
  ScrollWindowActivate()   scroll2.js line 293
  ScrollBarActivate()      scroll2.js line 440
  ButtonImageActivate()    scroll2.js line 535


There is NS4.7-specific code in the latter two functions. 
In ScrollBarActivate(): 

  if (is.ns4) this.lyrc.elm.captureEvents(Event.MOUSEDOWN |
                                          Event.MOUSEMOVE | Event.MOUSEUP)

In ButtonImageActivate():

  if (is.ns4) this.lyrc.elm.captureEvents(Event.MOUSEDOWN | Event.MOUSEUP)


I don't know if these are relevant to the problem. By setting 
|is.ns4| = true in the debugger, it seemed to make no difference.

Comment 7

[Phil Schwartau]

Attachment: scroll2.js

Comment 8

[Phil Schwartau]

Attachment: dynlayer.js

Comment 9

[Phil Schwartau]

My feeling is, this is DOM and not JS Engine. Or perhaps Evangelism.
Reassigning to DOM Level 0 for a look. 

cc'ing bclary to see if he knows if the DynAPI Scroll2 Object ever
works in Mozilla - 

Comment 10

[Bob Clary [:bc] (inactive)]

This is Dan Steinman's original DynAPI which is no longer being developed or
supported. I do not know if this code ever worked in Mozilla.

Comment 11

[Phil Schwartau]

From http://www.dansteinman.com/dynduo/en/scroll2.html

"This will likely be the last major upgrade to this code before NS5 comes out 
when I'll probably have to edit it a bit to make it work properly (haven't even 
tried it yet). The usefulness of this object will be an issue when NS5 is fully 
out because it will also have the IFrame tag by which you can bring in external 
files. But even still, this is a fun object to build some wild displays and 
break out of those ugly regular scrollbars."


From 
http://dynapi.sourceforge.net/doccenter/index.php?Dynamically%20Add%2FRemove%20S
crollbars 

Dynamically Add/Remove Scrollbars 

"In IE you can dynamically add/remove scrollbars. In NS you can also
dynamically add/remove scrollbars, but you can only remove them if the
content of the browser window IS actually smaller than the window. This is 
tested only on NS4.08 and IE5.5. NS6 ignores the code, but without errors."

Comment 12

[Phil Schwartau]

Attachment: scripts.js

Comment 13

[Phil Schwartau]

Reassigning to Evangelism component. If anyone disagrees with this,
please reassign back to DOM Level 0 for further debugging.

I can't pinpoint the exact spot in this version of DynAPI that is not
compliant with W3C or N6. It would be great if someone could pinpoint it.

The latest release of the DynAPI can be found at:
http://dynapi.sourceforge.net/doccenter/index.php?ReleaseNotes
 

Comment 14

[Phil Schwartau]

By the way, the scripts.js file is from http://www.pariscope.fr/scripts.js
and is not a DynAPI file. I included it in case there is any mistake there.

Increasing Severity to major because of the importance of this site.

Comment 15

[Phil Schwartau]

If you modify scroll2.js as follows, you can at least see the red 
scroll button in Mozilla: 


function ScrollActivate(w,h,reset) {

  if (!this.activated) {
    this.lyr=new DynLayer(this.name)

    if (this.useV) {
      this.barV.activate()
      this.up1Img.activate()
      this.dn1Img.activate()
      this.up2Img.activate()
      this.dn2Img.activate()
    }

    if (this.useH) {
      this.barH.activate()
      this.ltImg.activate()
      this.rtImg.activate()
    }
  }

  this.window.activate(w,h)

  if (this.useV) {

    if (reset!=false) this.barV.boxlyr.moveTo(null,0)
  
    ////////////if (!this.window.enableVScroll) this.barV.boxlyr.hide()
    ////////////else this.barV.boxlyr.css.visibility="inherit"
    this.barV.boxlyr.css.visibility="inherit" //<<<<<<<<<<<<<< i.e FORCE THIS! 
  }
  
  if (this.useH) {
    if (reset!=false) this.barH.boxlyr.moveTo(0,null)

    if (!this.window.enableHScroll) this.barH.boxlyr.hide()
    else this.barH.boxlyr.css.visibility="inherit"
  }
  
  this.activated=true
}


|this.useV| == true, which means there should be vertical scrolling.
However, I found that in Mozilla, |this.window.enableVScroll| == false,
which is why the red button doesn't show up in the vertical scrollbar.
So we have to force the line

              this.barV.boxlyr.css.visibility="inherit"

in order to see it. Once the button shows up, however, it responds to
mouse events in a strange fashion. Movement of the button can be delayed
as much as two or three seconds. The right mouse button has more effect
than the left. And the text to the left of the scrollbar never seems to move. 

This is probably due to the lack of Netscape6 code-branching in the 
mouse-events routines I outlined above at 2001-11-01 14:34.

Comment 16

[Tristan Nitot]

Accepting bug. This is definitely an evangelism issue, IMHO.

Setting to P2.
sent letter to [e-mail] and through the feedback form.

Comment 17

[Tristan Nitot]

see also http://bugzilla.mozilla.org/show_bug.cgi?id=87072

Comment 18

[Mathieu Pillard [:mat]]

There are 2 ways to fix this bug:
-1 tell them to use iframes. That will work for the scoll thingie, dunno if they
use dynapi elsewhere...
-2 tell them to use a fixed version of dynapi.
Either dynapi1 fixed by bob clary (http://www.bclary.com/dynapi/)
Or dynapi2 (http://dynapi.sourceforge.net/dynapi/) ...

Comment 19

[Pierre Chanial]

I emailed the webmaster and got an answer. Here is an excerpt:

> Le site pariscope.fr est en effet 
> difficilement visible sur certaines version de navigateurs (les plus récentes
> notement). Pour remedier à ce probleme, nous avons mis au point une version 
> texte sans ascenceurs internes aux pages.
> Elle est disponible à l'adresse suivante :
> http://txt.pariscope.fr
(snip) 
> la version graphique sera retouchée d'ici septembre pour supprimer les 
> ascenceurs !
> Cordialement,La webmaster de pariscope.fr

Good good good, Tristan, I guess you won't mind if I set the target to september
for you.

Comment 20

[Tristan Nitot]

Good job Pierre !

Comment 21

[Tristan Nitot]

*** Bug 87072 has been marked as a duplicate of this bug. ***

Comment 22

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

just a comment about this site : pariscope.fr has also a lot of problems with
IE6, not to mention the strange behavior of the dyn scrollbars...
So the site does not have problems with "some" navigators as stated the webmaster
but with all navigators. me wonders who's the contractor who did that for them.

Comment 23

[Mathieu Pillard [:mat]]

seems to be working now, the js scroll has gone... 

Comment 24

[Olivier Cahagne]

Shall we close it as WFM or FIXED ?

Comment 25

[Bob Clary [:bc] (inactive)]

since the site changed their content -> fixed. 

Comment 26

[Mathieu Pillard [:mat]]

marking as such.

Comment 27

[Peter Van der Beken [:peterv]]

Yay.

Comment 28

[Bob Clary [:bc] (inactive)]

tech evang june 2003 reorg