After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 460284 - Orca should not allow the user to arrow into the FF3 status bar
Orca should not allow the user to arrow into the FF3 status bar
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.19.x
Other All
: Normal normal
: 2.20.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404403 404409
 
 
Reported: 2007-07-25 16:05 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-07-22 19:28 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
proposed patch (603 bytes, patch)
2007-07-29 22:18 UTC, Joanmarie Diggs (IRC: joanie)
none Details | Review
take 2 (608 bytes, patch)
2007-08-05 00:49 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2007-07-25 16:05:13 UTC
On certain pages, something is convincing Orca that the next line of content in the page is the status bar.  When this occurs, arrowing down from the the last line of the page causes the status bar text to be read.  Arrowing down from there moves focus back to the last line of the page.  This gives one the impression that they are stuck rather than at the end of the page.  An example where this occurs is here: http://www.sun.com/desktop/index.jsp
Comment 1 Mike Pedersen 2007-07-25 16:23:18 UTC
This can also be reproduced when arrowing from the bottom of a html message in thunderbird.
Comment 2 Joanmarie Diggs (IRC: joanie) 2007-07-25 16:31:02 UTC
That's odd.... But true.  Thanks for pointing that out!
Comment 3 Joanmarie Diggs (IRC: joanie) 2007-07-29 22:18:58 UTC
Created attachment 92660 [details] [review]
proposed patch

Check that obj is in document content before returning it in findNextCaretInOrder().  Doing so seems to solve the problem for both Firefox and Thunderbird.  Mike please test.  Thanks!
Comment 4 Mike Pedersen 2007-08-03 21:12:11 UTC
This seems to work well.  It also fixes getting stuck on the "you are on-line" button in thunderbird.  The only thing left which is probably a different bug is that you can still get stuck in the "to" field if you up arrow past the top of a thunderbird message that you are reading. 
Comment 5 Joanmarie Diggs (IRC: joanie) 2007-08-03 21:32:31 UTC
Thanks for testing.   Before we declare it a different bug, I'll try to reproduce it.  The reason I say that is that it crossed my mind that we might have the very same issue with findPreviousCaretInOrder(), but I couldn't manage to up arrow out of the document frame.  I figured, why fix code that ain't broke?  Now I'm figuring that I should have tried harder with the up arrow. <grin>  Stay tuned....
Comment 6 Joanmarie Diggs (IRC: joanie) 2007-08-03 22:02:19 UTC
Okay, I can sometimes arrow into the date field of an HTML message.  I just put  the very same check into findPreviousCaretInOrder() and that didn't stop the behavior.  So something else is going on.  Still investigating...
Comment 7 Joanmarie Diggs (IRC: joanie) 2007-08-05 00:49:30 UTC
Created attachment 93094 [details] [review]
take 2

Perhaps I'm just tired.... But if obj is not in document content, find{Next,Previous}CaretInOrder() should always return None (right??).  Seems too easy....

Mike please test.  Will what am I missing?  Thanks guys!
Comment 8 Mike Pedersen 2007-08-06 04:19:33 UTC
I tested this with the Thunderbird from yesterday and it does seem to solve the problem I reported.
Comment 9 Willie Walker 2007-08-10 16:42:15 UTC
(In reply to comment #7)
> Created an attachment (id=93094) [edit]
> take 2
> 
> Perhaps I'm just tired.... But if obj is not in document content,
> find{Next,Previous}CaretInOrder() should always return None (right??).  Seems
> too easy....

I think find{Next,Previous}CaretInOrder() assume you're starting inside document content, so a check at the beginning for this precondition makes a lot of sense.
Comment 10 Joanmarie Diggs (IRC: joanie) 2007-08-11 07:19:27 UTC
Too easy AND makes a lot of sense.  I'm definitely scared now. ;-)

Patch committed to trunk and to the 2.20 branch.  Moving to [pending]. Thanks guys!
Comment 11 Mike Pedersen 2007-08-13 22:31:27 UTC
I've tested this one and it seems to work as expected.  
Comment 12 Joanmarie Diggs (IRC: joanie) 2007-08-13 22:38:54 UTC
Thanks Mike.  Closing as FIXED.