The default bug view has changed. See this FAQ.

Windows magnifier tool doesn't focus correctly when there is a text editing at firefox 4

RESOLVED FIXED in mozilla6

Status

()

Core
Disability Access APIs
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: fredy, Assigned: jimm)

Tracking

(Blocks: 1 bug, {regression, reproducible})

Trunk
mozilla6
x86
Windows XP
regression, reproducible
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:2.0.1) Gecko/20100101 Firefox/4.0.1
Build Identifier: Mozilla/5.0 (Windows NT 5.1; rv:2.0.1) Gecko/20100101 Firefox/4.0.1

The magnifier tool helps people with vision problems to magnify the screen and it is vital to focus properly. This doesn't happen when the magnifier option "Follow text editing" is on and there is text editing at firefox 4.

Reproducible: Always

Steps to Reproduce:
1. Open magnifier tool from Start menu-> Programs -> Accessories ->  Accessibility -> Magnifier.
2. Change the height of magnifier window so it can be viewed only one line.
3. Be sure that the option "Follow text editing" at the magnifiers settings is enabled (ticked).
4. Open firefox and click at URL address field at the navigation toolbar.
5. Start typing at the url field.
6. The magnifier window focus is now under this field and not at the field.

Actual Results:  
The focus at firefox 4 when there is text editing is under the text that it should be focused on.

Expected Results:  
The focus at firefox 4 when there is text editing to be at the text that is being edited.

If the "Follow text editing" is not selected (enabled/ticked) then the problem doesn't exist.

This problem occur not only at firefox user interface but also at web pages where you can edit text (like text form fields).
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: reproducible

Comment 1

6 years ago
Bug 648863?
Alice, I'm not 100% sure this is a dupe.

Alfredos-Panagiotis, thanks for filing. Can you try changing this in your about:config settings:
gfx.direct2d.disabled true

I'm curious if this has to do with the caret not automatically following our direct write calls.
(In reply to comment #2)
> Alice, I'm not 100% sure this is a dupe.
> 
> Alfredos-Panagiotis, thanks for filing. Can you try changing this in your
> about:config settings:
> gfx.direct2d.disabled true
> 
> I'm curious if this has to do with the caret not automatically following our
> direct write calls.

Either true or false the bug still exists. :(
Summary: Windows magnifier tool doesn't focus correctly when there is a text editing in firefox 4 → Windows magnifier tool doesn't focus correctly when there is a text editing at firefox 4
Alice it sounds to me the same or a too similar bug. I have tried safe mode that it is suggested at that other bug and I have not seen any difference.

Comment 5

6 years ago
Confirmed with disabled auto hide MenuBar on
http://hg.mozilla.org/mozilla-central/rev/e0f6db50231f
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0a1) Gecko/20110510 Firefox/6.0a1 ID:20110510030615

If Enable App Button(i.e enabled auto hide MenuBar), it works properly.

Comment 6

6 years ago
Regression window(cached m-c nightly):
Works:
http://hg.mozilla.org/mozilla-central/rev/522df66198cf
Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a6pre) Gecko/20100624 Minefield/3.7a6pre ID:20100624185700
Fails:
http://hg.mozilla.org/mozilla-central/rev/51bd519736c4
Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a6pre) Gecko/20100624 Minefield/3.7a6pre ID:20100624221810
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=522df66198cf&tochange=51bd519736c4
Blocks: 513162, 555081
Keywords: regression

Updated

6 years ago
Duplicate of this bug: 648863
(Assignee)

Comment 8

6 years ago
Hmm, maybe I've not done this right, but it seems to be working for me on Win7. I have "fallow the text insertion point" checked in the magnifier options. When I click on the address bar and type, the magnifier window fallows the cursor. Attaching a screen shot...
(Assignee)

Comment 9

6 years ago
Created attachment 531439 [details]
mag pic
(Assignee)

Comment 10

6 years ago
(In reply to comment #8)
> Hmm, maybe I've not done this right, but it seems to be working for me on
> Win7. I have "fallow the text insertion point" checked in the magnifier
> options. When I click on the address bar and type, the magnifier window
> fallows the cursor. Attaching a screen shot...

bleh: fallows -> follows. brain short circuit.
At windows XP is what Alice said that if you hide the menu bar and have the firefox orange button (App Button) everything works fine but if you have the menu bar then the bug shows up.

Comment 12

6 years ago
(In reply to comment #8)
> Hmm, maybe I've not done this right, but it seems to be working for me on
> Win7. I have "fallow the text insertion point" checked in the magnifier
> options. When I click on the address bar and type, the magnifier window
> fallows the cursor. Attaching a screen shot...

You should _Disable_ Auto Hide MenuBar. 
i.e, Check MenuBar in View > Toolbars > Check MenuBar
(Assignee)

Comment 13

6 years ago
(In reply to comment #12)
> (In reply to comment #8)
> > Hmm, maybe I've not done this right, but it seems to be working for me on
> > Win7. I have "fallow the text insertion point" checked in the magnifier
> > options. When I click on the address bar and type, the magnifier window
> > fallows the cursor. Attaching a screen shot...
> 
> You should _Disable_ Auto Hide MenuBar. 
> i.e, Check MenuBar in View > Toolbars > Check MenuBar

Ok, thanks Alice, I can reproduce.
(Assignee)

Comment 14

6 years ago
Hmm, not sure what the magnifier is doing here. The sdk accessibility inspector doesn't have any trouble matching up the bounding rect of text edits, so it doesn't appear to be an accessibility api problem.
(Assignee)

Comment 15

6 years ago
(In reply to comment #14)
> Hmm, not sure what the magnifier is doing here. The sdk accessibility
> inspector doesn't have any trouble matching up the bounding rect of text
> edits, so it doesn't appear to be an accessibility api problem.

Although, magnifier does appear to be using accessibility apis for it's information.

Comment 16

6 years ago
As mentioned already on Bug 648863 (I'd say it's exactly the same problem, I was not very clear in explaining) it helps to hide the menu bar in order to type, but the magnified part jumps somewhere else when pressing the arrow keys (most times).
(Assignee)

Comment 17

6 years ago
Hmm, height and width values returned by accLocation are correct. Not sure what's going on here.
(Assignee)

Comment 18

6 years ago
(In reply to comment #17)
> Hmm, height and width values returned by accLocation are correct. Not sure
> what's going on here.

X and Y positions on this call are correct as well, and the caret position we set via nsAccessibleWrap::UpdateSystemCaret() is also correct. So far it's a mystery.

Comment 19

6 years ago
In addition, this problem occurs with caret browsing mode too.
(Assignee)

Updated

6 years ago
No longer blocks: 555081
(Assignee)

Comment 20

6 years ago
Created attachment 534062 [details] [diff] [review]
fix v.1
Assignee: nobody → jmathies
Attachment #531439 - Attachment is obsolete: true
(Assignee)

Comment 21

6 years ago
Created attachment 534066 [details] [diff] [review]
fix v.1

This broke when we removed child widgets from the view. GetNearestWidget includes any chrome on the widget. In the past content widgets didn't have chrome, but now the top level widget does, so we have to subtract that off from WidgetToScreenOffset which calculates the content offset.

I could also simplify this by adding a new widget method for the window offset, or use GetScreenBounds. This patch makes the fewest changes though.
Attachment #534062 - Attachment is obsolete: true
Attachment #534066 - Flags: review?
(Assignee)

Updated

6 years ago
Attachment #534066 - Flags: review? → review?(bolterbugz)
(Assignee)

Comment 22

6 years ago
I missed a parenthesis in that comments, it should be:

// ((content screen origin) - (content offset in the widget)) = widget origin on the screen
Comment on attachment 534066 [details] [diff] [review]
fix v.1

Jim, david's going to be on vacation on Monday (Canadian holiday), so if we want to get this into Firefox 6 still, you might want to change the review request to either me or surkov. I looked at the patch, and this definitely looks like the right fix to me.

Updated

6 years ago
Status: NEW → ASSIGNED
Version: unspecified → Trunk
(Assignee)

Comment 24

6 years ago
Comment on attachment 534066 [details] [diff] [review]
fix v.1

all yours then!
Attachment #534066 - Flags: review?(bolterbugz) → review?(marco.zehe)
Comment on attachment 534066 [details] [diff] [review]
fix v.1

r=me, thanks for the patch!
Attachment #534066 - Flags: review?(marco.zehe) → review+
(Assignee)

Comment 26

6 years ago
http://hg.mozilla.org/mozilla-central/rev/6346ef40d7ab
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

6 years ago
Assignee: jmathies → nobody
Component: Disability Access → Disability Access APIs
Product: Firefox → Core
QA Contact: disability.access → accessibility-apis
Target Milestone: --- → mozilla6

Comment 27

6 years ago
any change to get mochitest for this?
Flags: in-testsuite?

Updated

6 years ago
Assignee: nobody → jmathies
(Assignee)

Comment 28

6 years ago
(In reply to comment #27)
> any change to get mochitest for this?

Might be possible but tricky I think since it relies on results are based on the desktop position of the browser window.

Updated

6 years ago
Duplicate of this bug: 670342

Comment 30

5 years ago
First, I am new here and not a developer, so sorry if I don't follow protical. 

It states that this issues is resolved, but I am still having issues with Microsoft Magnifier focusing on the caret in Firefox 4+. It will follow text editing well when the cursor is at the end of a section of text, but has issues when the cursor is anywhere else within the text. It TRIES to focus on the cursor, but is above the text by several lines. This occurs when you are as little as one character back from the end of the paragraph. Also, when trying to browse using the caret I experience the same issue. The magnifier focuses several lines above the caret. Any help would be appreciated. Thanks.
(Assignee)

Comment 31

5 years ago
(In reply to kdubfreak from comment #30)
> First, I am new here and not a developer, so sorry if I don't follow
> protical. 
> 
> It states that this issues is resolved, but I am still having issues with
> Microsoft Magnifier focusing on the caret in Firefox 4+. It will follow text
> editing well when the cursor is at the end of a section of text, but has
> issues when the cursor is anywhere else within the text. It TRIES to focus
> on the cursor, but is above the text by several lines. This occurs when you
> are as little as one character back from the end of the paragraph. Also,
> when trying to browse using the caret I experience the same issue. The
> magnifier focuses several lines above the caret. Any help would be
> appreciated. Thanks.

What version of Firefox are you using? The fix here went out with Fx 6, which was released around 2011-08-16.

Comment 32

5 years ago
I am using Firefox 9 and experience the problem with Windows XP at work and Windows 7 at home.
(Assignee)

Comment 33

5 years ago
(In reply to kdubfreak from comment #32)
> I am using Firefox 9 and experience the problem with Windows XP at work and
> Windows 7 at home.

Can you file a new bug please with steps to reproduce, and please cc me (:jimm).

Comment 34

5 years ago
(In reply to Jim Mathies [:jimm] from comment #33)
> (In reply to kdubfreak from comment #32)
> > I am using Firefox 9 and experience the problem with Windows XP at work and
> > Windows 7 at home.
> 
> Can you file a new bug please with steps to reproduce, and please cc me
> (:jimm).

Sure, it's my first time doing so, and I'll need to read up on the procedures and whatnot. Heading offline now. Will do so tomorrow and cc you.
(Assignee)

Updated

5 years ago
Blocks: 722178
You need to log in before you can comment on or make changes to this bug.