Cannot scroll

[pbmorton]

If the alert text cannot fit on the screen the text cannot be scrolled - UIAlertController in contrast supports scrolling by default. What's worse is that the PMAlertController places the "cancel" action off the screen so a user cannot then cancel the alert. The App is effectively blocked from user interaction.

Being able to handle an alert that wont fit completely on screen is important. With devices with different configurations (e.g. screen sizes, portrait, landscape) and dynamic type sizes (i.e. through the settings App a user can increase the text size) its easy to imagine some alerts not fitting a screen.

Otherwise a great library thanks!

[pmusolino]

Hi @pbmorton, great observation. I marked this issue like enhancement, i can implement this feature in a future release.

Thanks,
Paolo

[pbmorton]

Thanks Paolo. Can you estimate when this feature may be supported?

[pmusolino]

Asap, I'll try this week ;)

[javalnanda]

@codeido Have applied the quick workaround for the scroll issue. Do check if it looks good to you #45

[pmusolino]

@javalnanda thanks for this PR, but i think that your workaround is not a final solution. First of all, if you substitute the label description with a textview you break the back compatibility. This feature need to be an enhancement. 2nd: this solution include only some cases, because if you have a lot of buttons, you have a small textview for the description (probably with height == 0).

[javalnanda]

@codeido Makes sense, the back compatibility will be a major issue. Are you thinking of embedding entire alert view inside a scroll view?

[pmusolino]

Yes, i think that is the best solution.

[javalnanda]

ok, 👍 Let me know if you are already working on it. Else, I could spend some time on it the coming week.

[javalnanda]

@codeido Scrolling entire alert view looks really bad. Just tried out. The Default AlertController also scrolls only the text part. I think we will need to scroll only the content part except Alert Action Stack View

[javalnanda]

@codeido Sent one more PR, see if it is helpful. Also, I think it will be good to have a limit to the number of action buttons to max 5 or 6 maybe. Just a suggestion :)

[edwardmp]

Hi @codeido

thanks for this nice alert controller! Wondering if you ever got around the merging in a fix for this?

[thatmarcel]

@pmusolino When will the feature be implemented?

[sm0nster]

It's still unscrollable, right?