Closed Bug 518748 Opened 15 years ago Closed 15 years ago

send softkb notification for maemo

Categories

(Core :: Widget: Gtk, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
status1.9.2 --- beta1-fixed

People

(Reporter: blassey, Assigned: blassey)

Details

(Keywords: mobile)

Attachments

(1 file)

Attached patch patchSplinter Review
the maemo platform would benefit from receiving softkb notifications like WinMo does according to mfinkle.  This patch implements that
Attachment #402747 - Flags: review?(roc)
Comment on attachment 402747 [details] [diff] [review]
patch

String parsing sucks, seems like we should have a real interface for this...
Attachment #402747 - Flags: review?(roc) → review+
Assignee: nobody → bugmail
(In reply to comment #1)
> (From update of attachment 402747 [details] [diff] [review])
> String parsing sucks, seems like we should have a real interface for this...

There is nsPrintfCString, but that would require conversion to PRUnicode.  I figured the multiple appends were less sucky, but admittedly only slightly so.
I mean we shouldn't be serializing data to a string and then parsing it elsewhere.
Could we use nsIDOMRect or nsIDOMClientRect and pass that as the nsISupports parameter?
I don't know if that parameter is supposed to be used like that... maybe...
Attachment #402747 - Flags: approval1.9.2?
pushed http://hg.mozilla.org/mozilla-central/rev/dc57f5dc4fa3
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Attachment #402747 - Flags: approval1.9.2? → approval1.9.2+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: