forward arrow not active where it should be.

RESOLVED FIXED in mozilla1.0.1

Status

()

RESOLVED FIXED
17 years ago
11 years ago

People

(Reporter: thorgal, Assigned: radha)

Tracking

Trunk
mozilla1.0.1
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [adt3 RTM], URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

17 years ago
Mozilla 2002020611 (Linux/i686).

After clicking certain sequences on framed site, forward arrow of session
history misbehaves.

Steps to recreate problem:

1. Visit above URL.
2. Click java.lang in upper-left frame.
3. Click Math in lower-left frame.
4. Click a link to "E" in Field Summary table of main frame.
5. Move back in session history once (left arrow).
6. Observe that forward arrow (right) is not active.
7. Move back again, observe that forward arrow is now active and contains TWO items.

Step 6 is wrong, the arrow should be active.

Repeatibility: always.

Comment 1

17 years ago
wfm linux checkout finish: Thu Feb 14 16:50:59 GMT 2002
(Reporter)

Comment 2

17 years ago
Most interesting. With 2002021606/linux-i686 the problem is still there. Are you
sure you followed the "steps to recreate" correctly? Some other people on
#mozilla also found this bug to happen to them. Can someone else verify?
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0.1

Comment 3

17 years ago
same problem seen on http://slip/shrir/License.pdf (sorry folks, this is an 
internal url). I see the same above mentioned problem, FWD button does not 
highlight the very first time I try this url. You can try accessing any pdf 
online directly and reproduce this issue.
Keywords: nsbeta1
(Reporter)

Comment 4

17 years ago
You mean, with PDF plugin active?

Comment 6

17 years ago
Comment on attachment 82107 [details] [diff] [review]
patch to docshell

Radha, can you submit the patch again? This one is a dud.
Created attachment 82126 [details] [diff] [review]
This is a better one.
Attachment #82107 - Attachment is obsolete: true

Comment 8

17 years ago
Comment on attachment 82126 [details] [diff] [review]
This is a better one.

r=adamlock
Attachment #82126 - Flags: review+

Comment 9

17 years ago
Comment on attachment 82126 [details] [diff] [review]
This is a better one.

all these tweaks make me think that at some point we need to take a step back
and figure out why there are so many edge cases, and re-architect session
history.

in the mean time,
sr=alecf
Attachment #82126 - Flags: superreview+
Fixed in the trunk. 
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
ADT,  This fix has been in the trunk for a while. I would like it checked into
the branch as well. 
Keywords: adt1.0.1
greggl/radha - How visible, or prevelent do we believe this will be out in the
wild? Do we really believe a lot of users will hit this case?

Comment 13

17 years ago
adt1.0.1-
Keywords: adt1.0.1 → adt1.0.1-
Renominating for adt1.0.1. There is a fairly good chance of users hitting this
one, especially in sites using anchors extensively like this java documentation
site.   This is the third bug I have had  on this particular site.  The changes
are not massive either.  Please reconsider it for Mach V
Keywords: adt1.0.1- → adt1.0.1
(Reporter)

Comment 15

17 years ago
I support Radha on this one. The patch is simple and really useful. I've been
using it for a month with no problems at all.
this is still a minus. adt1.0.1- per adt
Keywords: adt1.0.1, nsbeta1 → adt1.0.1-, nsbeta1-
Whiteboard: [adt3 RTM]

Updated

11 years ago
Component: History: Session → Document Navigation
QA Contact: claudius → docshell
You need to log in before you can comment on or make changes to this bug.