Closed Bug 811177 Opened 11 years ago Closed 10 years ago

document designMode does not bring up the keyboard


(Firefox OS Graveyard :: Gaia::Keyboard, defect)

Not set


(Not tracked)



(Reporter: xyuan, Assigned: janjongboom)




(Whiteboard: [label:Keyboard & IME])


(3 files, 2 obsolete files)

If we set the designMode property to 'on', the entire document will be editable. The keyboard doesn't respond to such type of document.

Testcase provided Martijn Wargers [:mw22]:
 - http://people
See Also: → 796365
Whiteboard: [label:Keyboard & IME]
Can we fix this as soon as possible?  It is blocking the release of my app (
Another way to set the entire document editable is letting document.designMode = “on”, which can't bring keyboard as well.
Blocks: keyboard-api
Severity: minor → normal
Component: Gaia::System → Gaia::Keyboard
I'll take a look.
Assignee: nobody → janjongboom
Attached patch Gecko patch (obsolete) — Splinter Review
Here is a patch
Attachment #796100 - Flags: review?(xyuan)
Attached file Tests (obsolete) —
Attachment #796102 - Flags: review?(xyuan)
Will the patch break bug 902168?
Flags: needinfo?(janjongboom)
I don't think so, at least contenteditable kept working for me in both SMS as well as in the tests and I havent seen any messages, but it looks like it could break that yeah. I'll write a test case...
Flags: needinfo?(janjongboom)
I made a mistake in comment 2, and the other way to set the entire document editable should be:
 <body contenteditable>contentEditable</body>
On b2g-desktop, the following test failed with gecko patch of Comment 4:

1. Apply gaia_test_case.patch to gaia
2. Launch b2g-desktop and open |UI Tests| -> |Empty|
3. Click the content text 'Hello World!', which is editable

The keyboard is launched.

The keyboard won't show for the first time, but after open the keyboard with other input field and reproduce the above steps, the keyboard may show.
Attachment #796100 - Flags: review?(xyuan)
The bug addressed in Comment 9 has to do with nested iframes (which is what we have in UI tests)... Will take a look.
Attached patch Gecko patch v2Splinter Review
Hi Yuan, the attached patch passes:

* iframe document with designMode=on (tests attached)
* iframe with body contenteditable (bug 902168) (tests attached)
* test case provided in gaia_test_case.patch
Attachment #796100 - Attachment is obsolete: true
Attachment #796568 - Flags: review?(xyuan)
Comment on attachment 796568 [details] [diff] [review]
Gecko patch v2

Review of attachment 796568 [details] [diff] [review]:


Tested on device and b2g-desktop. The patch works fine.
Attachment #796568 - Flags: review?(xyuan) → review+
Comment on attachment 796102 [details] [review]


I don't actually run the test, but it looks good for me.

And could please also add a test page to UI tests app so that QA can help to verify the bug.
Attachment #796102 - Flags: review?(xyuan) → review+
Added iframe tests to UI tests, and landed in (needs gecko patch to have landed as well to be working of course)
Well done!
Depends on: 910761
Closed: 10 years ago
Resolution: --- → FIXED
(In reply to Jan Jongboom [:janjongboom] from comment #14)
> Added iframe tests to UI tests, and landed in
> 53d993416233fd1f5de3976ac49c62f5ddc465af (needs gecko patch to have landed
> as well to be working of course)

Backed out due to this depends on the Gaia patch of Bug 906096, which is also backed out for new IME framework to land.

revert commit:
Was not reopened when backing out...
Resolution: FIXED → ---
Attached file Tests (need relanding)
Attachment #796102 - Attachment is obsolete: true
Attachment #802180 - Flags: review+
Relanded in
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
Attachment mime type: text/plain → text/x-github-pull-request
Attachment mime type: text/plain → text/x-github-pull-request
See Also: → 1144069
You need to log in before you can comment on or make changes to this bug.