Closed
Bug 906506
Opened 11 years ago
Closed 11 years ago
cant write danish letters in the new version?
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox23 wontfix, firefox24 verified, firefox25 fixed, firefox26 verified, relnote-firefox 24+, fennec+)
VERIFIED
FIXED
Firefox 26
People
(Reporter: k.przybylak, Assigned: jchen)
References
Details
Attachments
(3 files)
120.40 KB,
image/png
|
Details | |
87.32 KB,
image/png
|
Details | |
3.51 KB,
patch
|
cpeterson
:
review+
bajaj
:
approval-mozilla-aurora+
bajaj
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
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 æøåÆØÅ
Comment 1•11 years ago
|
||
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•11 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)
Comment 3•11 years ago
|
||
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•11 years ago
|
Assignee: nobody → nchen
Assignee | ||
Comment 4•11 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•11 years ago
|
tracking-fennec: --- → ?
Comment 8•11 years ago
|
||
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
Comment 9•11 years ago
|
||
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•11 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•11 years ago
|
||
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 12•11 years ago
|
||
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•11 years ago
|
Keywords: checkin-needed
Comment 13•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/00be252820b7 Any way to test this?
Updated•11 years ago
|
tracking-fennec: ? → +
Comment 14•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/00be252820b7
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 26
Assignee | ||
Comment 15•11 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?
Comment 16•11 years ago
|
||
(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 17•11 years ago
|
||
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•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/613bd234d030
Updated•11 years ago
|
Comment 20•11 years ago
|
||
(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•11 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
Comment 22•11 years ago
|
||
Verified Fixed against the HTC One on latest m-b
Reporter | ||
Comment 23•11 years ago
|
||
working for me in 24 beta now
Reporter | ||
Comment 25•11 years ago
|
||
btw. you ask if I have try Nightly, it don't work om my device. I know that is off topic.
Comment 26•11 years ago
|
||
Worth release noting? Something like. Fixed an issue with non-Latin text entry with HTC keybord
relnote-firefox:
--- → ?
Comment 27•11 years ago
|
||
Ken you should be able to install Firefox Beta from the Play Store, it will have the fix as well.
Comment 28•11 years ago
|
||
(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.
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•