cant write danish letters in the new version?

VERIFIED FIXED in Firefox 24

Status

()

Firefox for Android
General
VERIFIED FIXED
4 years ago
10 months ago

People

(Reporter: Ken Przybylak, Assigned: jchen)

Tracking

24 Branch
Firefox 26
x86_64
Windows 8
Points:
---
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(firefox23 wontfix, firefox24 verified, firefox25 fixed, firefox26 verified, relnote-firefox 24+, fennec+)

Details

Attachments

(3 attachments)

(Reporter)

Description

4 years ago
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:23.0) Gecko/20100101 Firefox/23.0 (Beta/Release)
Build ID: 20130814063812

Steps to reproduce:

I think it is in the new vertion of fireforx for android, on webpages I can't write æøåÆØÅ anny more in inputbox. I have testet the beta whit same result.
I have tryed on version 23 and 24. (don't think I can download 25 and 26 that is in your list in bug report)


Actual results:

lettes don't apears


Expected results:

I can write æøåÆØÅ
Created attachment 791874 [details]
Firefox Beta (24) | Samsung Galaxy SIV (Android 4.3)

Which device and which keyboard? Which Android version? Have you tried Nightly for Android (http://nightly.mozilla.org)? Can you report back with that information and if Nightly works out for you?

Using Firefox Beta and testing out Nightly on my Samsung Galaxy S4 with a simple text input data:text/html,<input/> this works fine for me. I am using the stock Google Keyboard in Android 4.3

See the screenshot for sample input with character selection available on my keyboard.

Can you provide better steps with the information described?
Flags: needinfo?(k.przybylak)
(Reporter)

Comment 2

4 years ago
Hi There
I'm using Android 4.1.1 on a HTC one X.
I have now tested whit Firefox 23, beta and now nightly.
I'm using the stock keyboard QWERTY but have tested the number/phone keyboard whit same result.
The issue only appears in the body section not in address field.
I have tested HTML: <input> and <textarea>
I have no issue in crome or the other stock browser.
I can copy/paste æøå ind to the field but when pussing the butten nothing happens.
Regards
Kino
Flags: needinfo?(k.przybylak)
Created attachment 792208 [details]
Nightly (26) | LG Nexus 4 (Android 4.3)

I tried this out with the Dansk input language on the Google Keyboard and this worked out for me too (Android 4.3).
(Assignee)

Updated

4 years ago
Assignee: nobody → nchen
(Assignee)

Comment 4

4 years ago
Hi Ken, can you try this Nightly version? http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/nchen@mozilla.com-50499682bafd/try-android-x86/fennec-26.0a1.en-US.android-i386.apk

Thanks!
Assignee: nchen → nobody
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(k.przybylak)

Updated

4 years ago
Duplicate of this bug: 907110

Updated

4 years ago
tracking-fennec: --- → ?
Duplicate of this bug: 907405

Updated

4 years ago
Duplicate of this bug: 907405
Re-tested on my HTC One (seeing HTC as the commonality here) and was able to reproduce on Google. I enter accented characters such as àáâ, on insert the character is removed. I tested this on 24/25/26 (Android 4.2.2). Using system-language as English, and just invoking a long-tap on the letter to open a super menu with available accents.
status-firefox23: --- → affected
status-firefox24: --- → affected
status-firefox25: --- → affected
status-firefox26: --- → affected
Testing out Jim's try build fixed the issue for me (ARM-based build over at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/nchen@mozilla.com-50499682bafd/try-android/fennec-26.0a1.en-US.android-arm.apk)

Comment 10

4 years ago
(In reply to Aaron Train [:aaronmt] from comment #9)
> Testing out Jim's try build fixed the issue for me (ARM-based build over at
> http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/nchen@mozilla.com-
> 50499682bafd/try-android/fennec-26.0a1.en-US.android-arm.apk)

+1 on htc desire hd, android 2.3
(Assignee)

Comment 11

4 years ago
Created attachment 793525 [details] [diff] [review]
Properly commit onKeyMultiple() keys with unknown code (v1)

When onKeyMultiple() is called with KEYCODE_UNKNOWN, we're supposed to commit that text. The proper way to commit the text is to call BaseInputConnection.commitText() on the IC thread. We don't call GeckoInputConnection.commitText() because it can send key events which will cause onKeyMultiple() to be called again, producing a loop.
Attachment #793525 - Flags: review?(cpeterson)
Comment on attachment 793525 [details] [diff] [review]
Properly commit onKeyMultiple() keys with unknown code (v1)

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

LGTM!
Attachment #793525 - Flags: review?(cpeterson) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/00be252820b7

Any way to test this?
Assignee: nobody → nchen
Flags: in-testsuite?
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
tracking-fennec: ? → +
https://hg.mozilla.org/mozilla-central/rev/00be252820b7
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 26
(Assignee)

Comment 15

4 years ago
Comment on attachment 793525 [details] [diff] [review]
Properly commit onKeyMultiple() keys with unknown code (v1)

[Approval Request Comment]

Bug caused by (feature/regressing bug #): Bug 859452

User impact if declined: Cannot input any accented characters on HTC devices

Testing completed (on m-c, etc.): Locally; m-c

Risk to taking this patch (and alternatives if risky): Small; the code path is rarely used and only HTC devices should be affected

String or IDL/UUID changes made by this patch: None
Attachment #793525 - Flags: approval-mozilla-beta?
Attachment #793525 - Flags: approval-mozilla-aurora?
(In reply to Jim Chen [:jchen :nchen] from comment #15)
> Comment on attachment 793525 [details] [diff] [review]
> Properly commit onKeyMultiple() keys with unknown code (v1)
> 
> [Approval Request Comment]
> 
> Bug caused by (feature/regressing bug #): Bug 859452
> 
> User impact if declined: Cannot input any accented characters on HTC devices
> 
> Testing completed (on m-c, etc.): Locally; m-c
> 
> Risk to taking this patch (and alternatives if risky): Small; the code path
> is rarely used and only HTC devices should be affected

Please help QA with some testcases that may hit this codepath which can help them with some adhoc testing.Was the impact here limited to only Danish ? Would testing *only on* HTC devices be enough here?

> 
> String or IDL/UUID changes made by this patch: None
Comment on attachment 793525 [details] [diff] [review]
Properly commit onKeyMultiple() keys with unknown code (v1)

Request to land this asap so this can get into today's Beta which is going to build shortly.
Attachment #793525 - Flags: approval-mozilla-beta?
Attachment #793525 - Flags: approval-mozilla-beta+
Attachment #793525 - Flags: approval-mozilla-aurora?
Attachment #793525 - Flags: approval-mozilla-aurora+
(Assignee)

Comment 18

4 years ago
https://hg.mozilla.org/releases/mozilla-beta/rev/613bd234d030
status-firefox24: affected → fixed
status-firefox26: affected → fixed
status-firefox23: affected → wontfix
(Reporter)

Comment 19

4 years ago
I can test if you build a package.
Flags: needinfo?(k.przybylak)
(In reply to Jim Chen [:jchen :nchen] from comment #18)
> https://hg.mozilla.org/releases/mozilla-beta/rev/613bd234d030

Were you planning to push this to Aurora still?
(Assignee)

Comment 21

4 years ago
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #20)
> (In reply to Jim Chen [:jchen :nchen] from comment #18)
> > https://hg.mozilla.org/releases/mozilla-beta/rev/613bd234d030
> 
> Were you planning to push this to Aurora still?

Yes, pushed. I pushed to Beta early because the next Beta was going into build soon.

https://hg.mozilla.org/releases/mozilla-aurora/rev/7c2b0341e3de
status-firefox25: affected → fixed
Verified Fixed against the HTC One on latest m-b
Status: RESOLVED → VERIFIED
status-firefox24: fixed → verified
status-firefox26: fixed → verified
(Reporter)

Comment 23

4 years ago
working for me in 24 beta now
Duplicate of this bug: 910969
(Reporter)

Comment 25

4 years ago
btw. you ask if I have try Nightly, it don't work om my device. I know that is off topic.
Worth release noting? Something like.

Fixed an issue with non-Latin text entry with HTC keybord
relnote-firefox: --- → ?
Ken you should be able to install Firefox Beta from the Play Store, it will have the fix as well.
(In reply to Kevin Brosnan [:kbrosnan] from comment #26)
> Worth release noting? Something like.
> 
> Fixed an issue with non-Latin text entry with HTC keybord

https://www-dev.allizom.org/en-US/mobile/24.0/releasenotes/ .Added it to the release notes, let me know if it needs any edit's.
relnote-firefox: ? → 24+
You need to log in before you can comment on or make changes to this bug.