8927 - Link to anchor will not scroll overflow data.

Status: VERIFIED DUPLICATE of bug 62536

(Core :: Layout, defect, P2)

Description

[webmas7]

There are two links on this page. One is the image. The other is the visable
link. There is an anchor at the end of the list, where the data is not visable
without scrolling the box manually.

Neither link will correctly auto-scroll to the end of the overflow data so the
end of the data is visable.

Comment 1

[rickg]

Chris -- The point of this report is not clear to me. Can you get back to the
author and get the problem stated more clearly. Note that under Gecko the
scrollable area is visible (where it is not under Nav4).

Comment 2

[webmas7]

Description #2

In the area of the overflow data, I have a named anchor way at the bottom to
where a person would have to scroll (the box) to see it. I have a link to the
named anchor elsewhere in the document.

Now, clicking on the link to the named anchor should automatically scroll the
box down the where the anchor is in the overflow data. It does not. See IE5, as
it works.

Hopefully this is more clear?

Thanks.

Eric

Comment 3

[Chris Petersen]

I can reproduce this problem. I have attached a simplified version of the URL
above. The text is contained in DIV element that has been assigned a
overflow:auto css property. A link at the top of the document (Travel|Tourism)
references a anchor at the bottom of the document.Clicking on the top link
dosen't scroll to the bottom.

Comment 4

[Chris Petersen]

Attachment: a DIV element (assigned OVERFLOW) containing a HREF anchor

Comment 5

[rickg]

It sounds like the problem is that the document is not autoscrolling to the
right place when a link (near the top) is clicked, which should reveal an anchor
lower (in the scrollable area). Tom, please take a look.

Comment 6

[joki (gone)]

Bulk prioritizing bugs for correct milestones.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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

Comment 8

[ckritzer (gone)]

Nominating for nsbeta3 because of Standards Compliance.

Comment 9

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I am the virtual joki.

Comment 10

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Per discusion with Nisheeth, marking nsbeta3+. Will email ekrock to verify.

Comment 11

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: First cut fix, we were using wrong scrolling view

Comment 12

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: Twisted testcase: scroll inside scroll

Comment 13

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Since this isn't event handling, and Chris Waterson was kind enough to accept 
this, I'll reassign...

The problem is in PresShell::ScrollFrameIntoView. Right now it takes the root 
scrollable view and sees if it needs to scroll that to get the frame into view. 
This breaks when there are several scrollable views one inside the other.

My first cut fix fixes the problem in the first test case. However, it breaks in 
the second (the twisted test case). What still needs to happen is that we need 
to scroll the scrollable views too.

IE5 gets this right.

Comment 14

[James Lariviere]

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

Comment 15

[Hixie (not reading bugmail)]

Upon managerial request, adding the "testcase" keyword to 84 open layout bugs that
do not have the "testcase" keyword and yet have an attachement with the word
"test" in the description field. Apologies for any mistakes.

Comment 16

[alex.dent]

i don't think the target milestone should be the undefinded future, as this bug
makes some otherwise very nice constructs useless.

can the target milestone be updated? or is it a very non-trivial thing?

btw, this is not a pc/win98 specific bug.

Comment 17

[Greg K.]

Confirmed using Mac-2001101117, and so marking All/All.

Comment 18

[basic]

with build 2001111903 win32 this seems to wfm

Comment 19

[basic]

both attachment 575 [details] and attachment 13128 [details] wfm with build 2001112104 win32 on w2k

Comment 20

[basic]

but something weird is definitely going on in attachment 13128 [details]. Seems to me that
the page does not layout correctly after reloading a few times.

Comment 21

[Heikki Toivonen (remove -bugzilla when emailing directly)]

jst said he fixed this (he did not know this bug existed), and I tested the
twisted testcase. Seems to work. jst, can you dupe this to the one you fixed?

Comment 22

[Johnny Stenback (:jst)]

Yeah, the bug where I fixed ScrollFrameIntoView() was bug 62536, duping.

*** This bug has been marked as a duplicate of 62536 ***

Comment 23

[Matt Perry]

v