Closed Bug 980714 Opened 9 years ago Closed 9 years ago
Blinking caret in panel text
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:27.0) Gecko/20100101 Firefox/27.0 (Beta/Release) Build ID: 20140212131424 Steps to reproduce: Created a panel with a span, a div, and an unordered list with list items. Then clicked on each. Actual results: Blinking cursors appeared wherever I clicked. Expected results: Nothing. I tested the same HTML page in a tab and no cursors appeared.
Priority: -- → P2
Can confirm this not very nice right now. The problem is also reported here on the forum https://forums.mozilla.org/addons/viewtopic.php?f=27&t=15678&hilit=caret
An example of this problem can be seen if you install https://addons.mozilla.org/en-US/firefox/addon/heartbleed_monitor/, open the panel, and click near the "Settings" checkboxes.
Thanks Garrett, now its even easier for the developers to reproduce this problem. Same problem on OS X.
It seems the behavior was introduced adding the `showcaret` attribute. Not sure why it was added; Irakli do you recall anything about that? It seems that text inputs have caret even without this attribute.
Attachment #8407702 - Flags: review?(rFobic)
Attachment #8407702 - Flags: review?(rFobic) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/7e1e003219d18011df237b472d18ad21cd42344c Bug 980714 - Blinking caret in panel text - removed `showcaret` attribute https://github.com/mozilla/addon-sdk/commit/922dafe37147a8ae537b1c8b68352d15ad2c8784 Merge pull request #1464 from ZER0/panel-caret/980714 fix Bug 980714 - Blinking caret in panel text r=@gozala
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Can we get this uplifted to aurora and beta? It is the cause of frequent complaints in our addon which uses the SDK panel (https://github.com/EFForg/privacybadgerfirefox).
(In reply to Garrett Robinson [:grobinson] from comment #7) > Can we get this uplifted to aurora and beta? It is the cause of frequent > complaints in our addon which uses the SDK panel > (https://github.com/EFForg/privacybadgerfirefox). We're working on uplift patches, I plan to ask that this be uplifted to 30 & 31 ( you can test the fix in the current nightly build: http://nightly.mozilla.org/ )
[Approval Request Comment] Bug caused by (feature/regressing bug #): panel refactoring User impact if declined: users will see the blinking caret on the panel's content, even if the content is not editable. Testing completed (on m-c, etc.): m-c, m-a, locally Risk to taking this patch (and alternatives if risky): very low risk, we just removed the "showcaret" attribute. String or IDL/UUID changes made by this patch: none
Attachment #8426253 - Flags: approval-mozilla-beta?
Aurora seems have already the patch applied, see: http://mxr.mozilla.org/mozilla-aurora/source/addon-sdk/source/lib/sdk/panel/utils.js#223
This landed in github in the 31 cycle, per comment 6, so I assume it made it to trunk in 31 as well.
(In reply to Matteo Ferretti [:matteo] [:zer0] from comment #9) > [Approval Request Comment] > Bug caused by (feature/regressing bug #): panel refactoring When did this land? Is 29 or earlier affected?
Attachment #8426253 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Zero - can you answer Gavin's questions? Gavin: I think this only affects 29.
I think this was at least in FF 28 too probably even earlier but not sure.
(In reply to :Gavin Sharp (email firstname.lastname@example.org) from comment #12) > (In reply to Matteo Ferretti [:matteo] [:zer0] from comment #9) > > [Approval Request Comment] > > Bug caused by (feature/regressing bug #): panel refactoring > > When did this land? Is 29 or earlier affected? Yes, 29 and 28 are definitely affected. If our tags in github are correct, it's probably a bug we have since the panel refactoring, that it seems happened in Firefox 23.
Reproduced the initial issue on old Nightly (2014-03-06), verified using Heartbleed Monitor add-on on Firefox 30 beta 7 and latest Aurora on Windows 7 64bit, Mac OS X 10.9.2 and Ubuntu 14.04 32bit.
You need to log in before you can comment on or make changes to this bug.