151465 - iiyama.nl - Scroll bar doesn't scroll

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: Dutch, defect, P3)

Description

[Jaco Wolhuter]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.1a) Gecko/20020611
BuildID:    2002061104

I have seen this kind of error on more pages.  I think it is javascript related.
 The scroll bar moves, but the corresponding data doesn't scroll.

Reproducible: Always
Steps to Reproduce:
1.just scroll
2.
3.

Actual Results:  nothing

Expected Results:  scrolled

Comment 1

[Phil Schwartau]

Confirming bug using Mozilla trunk binary 2002061208 WinNT.

The JS source for this DHTML scrollbar is from
view-source:http://www.iiyama.nl/1024x768/scroll/scrollbar.js

The problem lies in this function, on the indicated line:

initScroller()
{
  if(ns6) pageFromTop = parseInt(tempLyr.style.top);



Note |tempLyr| is a <div> element in the middle frame at the site:
http://www.iiyama.nl/1024x768/php/quickselect-crt.php?search=1


                <div ID="main" class="main">


Indeed, in the Mozilla JS Debugger, we find:

tempLyr
[HTMLDivElement] [class: HTMLDivElement] {0}
tempLyr.style
[CSSStyleDeclaration] [class: CSSStyleDeclaration] {0}
tempLyr.style.top
[string] ""


Now, since |tempLyr.style.top| is the empty string, the line we
indicated above sets |pageFromTop| to parseInt("") = NaN:

pageFromTop
[double] NaN


This causes all subsequent arithmetic using |pageFromTop| to fail,
hence the scrollbar fails. You can see this directly by loading
the site in Mozilla; then before trying to scroll the middle frame,
copy this javascript:URL and hit enter

      javascript: void(window.frames[1].pageFromTop = 0);


This gives |pageFromTop| the value it has in IE on initial load.
Now try scrolling the middle frame in Mozilla: it works!

Comment 2

[Phil Schwartau]

Reassigning to DOM Level 0 for further analysis:

1. Is this just a coding error in the DHTML scrollbar?

2. Or should we be providing the numerical value 0 for |style.top|
   of the <div> element, instead of the empty string as we do?

Comment 3

[Boris Zbarsky [:bzbarsky]]

This is an error.  The "style" property should give a view onto the "style"
attribute of the object.  Since the "style" attribute does not set "top"
(indeed, it sets nothing), .style.top should be the empty string.

What the author wants here is not .style.top but the computed value of "top"
(which would take into account the stylesheet given by the <style> element.

Comment 4

[Phil Schwartau]

> What the author wants here is not .style.top but the computed value of "top"
> (which would take into account the stylesheet given by the <style> element.

Boris: since I don't know CSS very well, let me ask, How is that done?
How does the author access the computed value of "top"?

Thanks -

Comment 5

[Boris Zbarsky [:bzbarsky]]

document.defaultView.getComputedStyle(tempLyr, "").getPropertyValue("top")

a little verbose, I know.... At some point, we will support |.top| instead of
|.getPropertyValue("top")|...

Comment 6

[Mathieu Pillard [:mat]]

they need to use getComputedStyle to get the "real" top.

Comment 7

[Frederic Bezies]

Contact : [e-mail]

Comment 8

[Erik S-O Johansson]

I think they've updated the page, now WFM at Mozilla/5.0 (Windows; U; Windows NT
5.0; en-US; rv:1.2a) Gecko/20020910

Comment 9

[Mathieu Pillard [:mat]]

does work with mozilla 1.2a/win2k
DOESNT work with mozilla 1.0/linux . anyone can do some more testing ?

Comment 10

[Patrick]

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

Comment 11

[Patrick]

Using Moz 2003050622 on Mandrake 9.1 the problem is still there.

Comment 12

[Kai Lahmann (is there, where MNG is)]

move...

Comment 13

[Patrick]

June 2003 : prioritizing Dutch Tech evangs

Comment 14

[Andreas Kunz]

I see no scrollbar on that page now, but using the mousewheel does scroll
(WinXP, 1.7a)

Comment 15

[Patrick]

Attachment: Screenshot showing the "scrollbar"

(In reply to comment #14)
> I see no scrollbar on that page now, but using the mousewheel does scroll
> (WinXP, 1.7a)

They use their own customized "scrollbar": the arrow on the right for UP and
the two circular thingies for the current relative location.

Comment 16

[Andreas Kunz]

Ugh: this "scrollbar" is only displayed when the screen has a certain size; I
had to maximize Moz to see it. And it mostly works on XP (dragging the slider
quick and far does not), but this is no news since comment #8 and #9.

Comment 17

[Cyrille VIVION]

[bug248549notfixed]

Comment 18

[Patrick]

The site's been updated. Page no longer exist and the new iiyama pages seem to
work fine for me.