775714 - Metrofx should display the link click cursor when hovering over a link with the mouse

Status: RESOLVED FIXED

(Firefox for Metro Graveyard :: General, defect)

Description

[Jim Mathies [:jimm]]

Not sure how this is controlled. When the user interacts with the screen using touch, it looks like metro hides the mouse cursor automatically, which is convenient. However when moving around a page with the mouse, we only have the standard cursor displayed. It would be nice if we could get standard cursors working when mouse input is detected.

str:

1) open up a page with links
2) use the mouse to hover over a link

result: arrow mouse cursor continues to display while hovering.

Comment 1

[Jim Mathies [:jimm]]

edit: "we only have the *arrow* cursor displayed.."

Comment 2

[Jonathan Wilde [:jwilde]]

This is directly related to how mouse position events (mainly mousemove) aren't currently being messaged over to browser content.  Posting more details in related bug.

Comment 3

[Jim Mathies [:jimm]]

Prior to the elm landing in bug 776504 today we didn't have support for setting cursors. We did have some code in the init of the main view that set the cursor to the arrow. With cursor support turned on, platform (correctly I believe) set the cursor to eCursor_none resulting in no pointer.

So as a temporary measure, I've shunted the clear cursor call in the patch I landed:

https://hg.mozilla.org/projects/elm/rev/beed138536f0

See FrameworkView::ClearCursor().

Before testing front end work to get cursors displaying, that return statement needs to be removed. Also the SetCursor call in SetWindow can be removed as well.

Comment 4

[Jim Mathies [:jimm]]

With metro.debug.treatmouseastouch set to false, this now works in desktop mode. However there's still something wrong in the winrt backend with cursors. The event state manager seems to think it doesn't support setting cursors. I haven't found the root cause yet though.

Comment 5

[Jim Mathies [:jimm]]

here's the problem:

http://mxr.mozilla.org/mozilla-central/source/mobile/xul/chrome/content/cursor.css#9

Comment 6

[Jim Mathies [:jimm]]

https://hg.mozilla.org/projects/elm/rev/0e68def35666

Comment 7

[Matt Brubeck (:mbrubeck)]

Resolving bugs in the Firefox for Metro product that are fixed on the elm branch.  Sorry for the bugspam.  Search your email for "bugspam-elm" if you want to find and delete all of these messages at once.