Skip to content

new GrowingFormFieldType TEXTAREA#9

Open
caspahouzer wants to merge 7 commits intolambus-platform:masterfrom
caspahouzer:master
Open

new GrowingFormFieldType TEXTAREA#9
caspahouzer wants to merge 7 commits intolambus-platform:masterfrom
caspahouzer:master

Conversation

@caspahouzer
Copy link
Contributor

No description provided.

Copy link
Member

@hansemannn hansemannn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor change

index.js Outdated

const GrowingFormFieldType = {
TEXT: 'text',
TEXTAREA: 'textarea',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

camel case

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modified

Field types are superfluous, as they can be controlled via keyboardType.

Camel case textArea

Auto-focus next field via option focusNexField: true
@caspahouzer
Copy link
Contributor Author

README example updated

added 'return' listener for textfields for quicker input
const GrowingFormFieldType = {
TEXT: 'text',
EMAIL: 'email',
NUMBER: 'number',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this removed?

textField.keyboardType = Ti.UI.KEYBOARD_TYPE_DEFAULT;
break;
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restore all references to existing field types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants