Closed
Bug 8927
Opened 26 years ago
Closed 23 years ago
Link to anchor will not scroll overflow data.
Categories
(Core :: Layout, defect, P2)
Core
Layout
Tracking
()
Future
People
(Reporter: webmas7, Assigned: waterson)
References
()
Details
(Keywords: testcase, Whiteboard: [nsbeta3-][fixed])
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/html
|
Details |
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.
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).
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
Updated•26 years ago
|
Assignee: petersen → rickg
Comment 3•26 years ago
|
||
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•26 years ago
|
||
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.
Updated•25 years ago
|
Target Milestone: M16
I am the virtual joki.
Assignee: joki → heikki
Updated•25 years ago
|
Status: NEW → ASSIGNED
Per discusion with Nisheeth, marking nsbeta3+. Will email ekrock to verify.
Whiteboard: [nsbeta3+]
Updated•25 years ago
|
Priority: P3 → P2
Updated•25 years ago
|
Whiteboard: [nsbeta3+] → [nsbeta3+][fixinhand, check mlks etc.]
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.
Assignee: heikki → waterson
Status: ASSIGNED → NEW
Whiteboard: [nsbeta3+][fixinhand, check mlks etc.] → [nsbeta3+]
Assignee | ||
Updated•25 years ago
|
Whiteboard: [nsbeta3+]
Updated•24 years ago
|
Whiteboard: [need info]
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [need info] → [nsbeta3-]
Target Milestone: M18 → Future
Comment 14•24 years ago
|
||
*** Bug 50743 has been marked as a duplicate of this bug. ***
Comment 15•24 years ago
|
||
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.
Keywords: testcase
Comment 16•24 years ago
|
||
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•23 years ago
|
||
Confirmed using Mac-2001101117, and so marking All/All.
OS: Windows 98 → All
Hardware: PC → All
Comment 18•23 years ago
|
||
with build 2001111903 win32 this seems to wfm
Attachment #13128 -
Attachment is patch: false
Attachment #13128 -
Attachment mime type: text/plain → text/html
Comment 19•23 years ago
|
||
both attachment 575 [details] and attachment 13128 [details] wfm with build 2001112104 win32 on w2k
Comment 20•23 years ago
|
||
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.
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?
Whiteboard: [nsbeta3-] → [nsbeta3-][fixed]
Comment 22•23 years ago
|
||
Yeah, the bug where I fixed ScrollFrameIntoView() was bug 62536, duping.
*** This bug has been marked as a duplicate of 62536 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•