baqir123 / gtalksms

Automatically exported from code.google.com/p/gtalksms
0 stars 0 forks source link

Preferences fields need inputTypes #202

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Certain of the fields in the preferences could do with an inputType. 
(http://developer.android.com/reference/android/widget/TextView.html#attr_androi
d:inputType)
This would benefit the user by providing the appropriate keyboard for entering 
certain inputs.

The ones I noticed were all in res/xml/prefs_connection.xml:

notifiedAddress is an email address, so it should have a 
android:inputType="textEmailAddress".
login maybe needs the email input type, too? I use apps, so it's an email 
address on mine, but I'm not sure it's appropriate in general.
serverHost, serviceName and mucServer are domains, so they should probably have 
android:inputType="textUri".

Original issue reported on code.google.com by Eckankar on 24 Jul 2011 at 2:24

GoogleCodeExporter commented 9 years ago
Good point, we'll incorporate that. Or maybe you want to supply a patch?

Original comment by fschm...@gmail.com on 24 Jul 2011 at 6:29

GoogleCodeExporter commented 9 years ago
Here's a patch of the proposed changes.

I do not currently have a functioning Android SDK handy, so this is untested.

Also, it does not include the inputType for the login field, as I'm unsure as 
to whether it should have the textEmailAddress type. Opinions?

Original comment by Eckankar on 24 Jul 2011 at 8:01

Attachments:

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 4796425e39ee.

Original comment by fschm...@gmail.com on 19 Aug 2011 at 9:57

GoogleCodeExporter commented 9 years ago

Original comment by fschm...@gmail.com on 19 Aug 2011 at 9:59

GoogleCodeExporter commented 9 years ago

Original comment by fschm...@gmail.com on 25 Aug 2011 at 7:54

GoogleCodeExporter commented 9 years ago
Fixed in version 3.2

Original comment by Florent....@gmail.com on 28 Aug 2011 at 3:46