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

Support long text #9

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

afehners
Copy link

If the text is greater than 5 lines it gets truncated.
This switches from using a UILabel to a UITextView. If the text is longer than 5 lines it can now be scrolled, whilst still limiting the size of the alert, and allowing the alert to remain small for short text.

afehners added 2 commits May 31, 2013 13:04
If the text is greater than 5 lines it gets truncated.
This switches from using a UILabel to a UITextView. If the text is longer than 5 lines it can now be scrolled, whilst still limiting the size of the alert, and allowing the alert to remain small for short text.
Missed setting UITextView editing off
@ricsantos
Copy link
Contributor

In SIAlertView.m there is a constant #define MESSAGE_MAX_LINE_COUNT 5 - you can change that to make the alert show more lines of text.

@afehners
Copy link
Author

This is true. But set it to a high number and you will notice the popup box will grow larger than the bounding window causing the action buttons to disappear off the bottom of the screen.
This push request enables the defined number of lines to be shown, keeping the popup size limited, whilst allow large text to be scrolled like the built in UIAlertView

@filipengberg
Copy link
Contributor

+1

2 similar comments
@mattijsf
Copy link

mattijsf commented Feb 2, 2014

+1

@lanbozhang
Copy link

+1

@lanbozhang
Copy link

can this change request be included in master and next version?

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.

5 participants