newlines are displayed as spaces in landscape "full screen" textarea editor (Samsung keyboard)

RESOLVED FIXED in Firefox 23

Status

()

--
minor
RESOLVED FIXED
6 years ago
2 years ago

People

(Reporter: jasa.david, Assigned: jchen)

Tracking

22 Branch
Firefox 23
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
newlines are displayed as spaces in landscape "full screen" textarea editor (Samsung keyboard)

Firefox: 19.0, 22.0a1 (2013-02-22)
OS: android 4.1.2
device: Samsung galaxy s iii mini (i8190)

The bug occurs consistently on all sites in firefox. I also checked opera mobile and it doesn't suffer from the bug.

Updated

6 years ago
Keywords: testcase-wanted
This is screen size dependent. On my G2 I get the full screen keyboard. I can reproduce the behavior however the text does end up correctly displayed once you exit the fullscreen keyboard. Gmail also uses the fullscreen keyboard however they use the multiline fullscreen keyboard.
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: testcase-wanted
Hardware: Other → ARM
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Created attachment 732068 [details] [diff] [review]
Pass textarea input type (v1)

Use "textarea" as input type
Attachment #732068 - Flags: review?(masayuki)
Created attachment 732069 [details] [diff] [review]
Detect textarea and fix up other input type flags (v1)

Specify multiline flag for textarea and contentEditable/designMode documents

I also went ahead and reformatted the other lines
Attachment #732069 - Flags: review?(cpeterson)
Comment on attachment 732069 [details] [diff] [review]
Detect textarea and fix up other input type flags (v1)

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

LGTM.

Please file a follow-up bug to handle mIMEModeHint for input types "textarea", "search", and (if applicable?) contenteditable. We should also add an explicit case for mIMEModeHint.equals("text") instead of relying on the final `else` to implicitly handle "text".
Attachment #732069 - Flags: review?(cpeterson) → review+
Created attachment 732108 [details] [diff] [review]
Detect textarea (v2)

Patch to Just add textarea support. Going to refactor the rest of the code later.
Attachment #732069 - Attachment is obsolete: true
Attachment #732108 - Flags: review?(cpeterson)
Comment on attachment 732108 [details] [diff] [review]
Detect textarea (v2)

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

+1
Attachment #732108 - Flags: review?(cpeterson) → review+
Comment on attachment 732068 [details] [diff] [review]
Pass textarea input type (v1)

Sorry for the delay.
Attachment #732068 - Flags: review?(masayuki) → review+
https://hg.mozilla.org/mozilla-central/rev/0b5b5d9a8761
https://hg.mozilla.org/mozilla-central/rev/859e776239bf
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
You need to log in before you can comment on or make changes to this bug.