736879 - Hovering in the text area, but not directly on any text --> No text cursor

Status: RESOLVED FIXED

(DevTools :: Source Editor, defect, P3)

Description

[Dave Hennings]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20100101 Firefox/11.0
Build ID: 20120312181643

Steps to reproduce:

1. Opened the Scratchpad, by pressing shift+f4.
2. Moved the mouse pointer into the text area of the Scratchpad, but not onto any text


Actual results:

Nothing special.


Expected results:

The cursor should have changed into the "text cursor" (CSS cursor:text;)

More information:
The bug occurs for anything I know that uses Orion: the Scratchpad, the Style Editor and the Stylish extension.

Comment 1

[Rob Campbell [:rc] (:robcee)]

This is also consistent with the way our cursor changes when hovering text in the browser. Going to mark this WFM.

Comment 2

[Dave Hennings]

(In reply to Rob Campbell [:rc] (:robcee) from comment #1)
> This is also consistent with the way our cursor changes when hovering text
> in the browser. Going to mark this WFM.

Yes, it is "consistent" with the way our cursor changes when hovering text in the browser. Who cares? It's not supposed to be, IMO. It is inconsistent with <input type="text"> and <textarea> (which is sort of emulated) elements, and in fact any other text editor I know. I can't believe it won't be fixed when it seems like this is sufficient for solving the issue:

.viewContainer {
    cursor:text;
}

I am actually bothered by the current behavior: The constant blinking of the cursor as it switches between normal and text looks is annoying, and confusing.

Comment 3

[Rob Campbell [:rc] (:robcee)]

That's fine. If it's an easy enough CSS fix we can do it. This may require an upstream fix to Orion.

Moving to Source Editor.

Comment 4

[Anton Kovalyov (:anton)]

Attachment: Set cursor:text for the text area

Comment 5

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8338201 [details] [diff] [review]
Set cursor:text for the text area

Review of attachment 8338201 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!

Comment 6

[Anton Kovalyov (:anton)]

https://hg.mozilla.org/integration/fx-team/rev/f0cc82e94cce

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f0cc82e94cce

Comment 8

[svbabukumar]

 [bugday-20140305]

Validated the bug in Beta 28, Aurora 29 and Nightly 30 - Win 7 and it works fine