Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TIMOB-7739 add label for prompt and implement background properties #4409

Conversation

bijupmb
Copy link
Contributor

@bijupmb bijupmb commented Jun 24, 2013

String strPrompt = TiConvert.toString(d, "prompt");
promptText.setText(strPrompt);
}
if (d.containsKey("backgroundImage")) {
Copy link
Contributor

Choose a reason for hiding this comment

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

All these strings should be constants in TiC class. Some of these probably already exist there.

@ayeung
Copy link
Contributor

ayeung commented Jun 30, 2013

There needs to be a test case added to the ticket.

@ghost ghost assigned vishalduggal Jul 1, 2013
layout.setPadding(0,0,0,0);
layout.setPadding(0, 0, 0, 0);
int promptWidth = (int) TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, 150, proxy.getActivity()
.getResources().getDisplayMetrics());
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we restricting width to be 150 dp? You've already set it to single line and truncate at end. Why not FILL?

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

@vishalduggal
Copy link
Contributor

Needs a doc update and remerging with master.

@hieupham007
Copy link
Contributor

Please merge with master.

@hieupham007
Copy link
Contributor

Any update on this?

@bijupmb
Copy link
Contributor Author

bijupmb commented Sep 19, 2013

NEW PR:-
#4675

@vishalduggal
Copy link
Contributor

Closing this PR since new PR is up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants