'No, thanks' is in Bold and at the bottom #61

Closed
opened 2025-12-29 18:28:49 +01:00 by adam · 2 comments
Owner

Originally created by @ArturoLee on GitHub (Jul 17, 2017).

It seems that the 'No, thanks' AlertAction is in bold and is the last button on the AlertController. This is weird because the screen shots in library have 'No, thanks' as the first button, 'Remind me later' second, and 'Rate' last.

To me it makes more sense to have 'Rate' be the last button on the AlertController (like the screenshot).

We can just set the style for the Rate AlertAction to UIAlertActionStyle.cancel
And also set the style for the Don't Rate AlertAction to UIAlertActionStyle.default

This will basically make the AlertController look like the one in the Armchair screenshots.

Is there a reason it's not like this by default? Thank you!

Originally created by @ArturoLee on GitHub (Jul 17, 2017). It seems that the 'No, thanks' AlertAction is in bold and is the last button on the AlertController. This is weird because the screen shots in library have 'No, thanks' as the first button, 'Remind me later' second, and 'Rate' last. To me it makes more sense to have 'Rate' be the last button on the AlertController (like the screenshot). We can just set the style for the **Rate** AlertAction to **UIAlertActionStyle.cancel** And also set the style for the **Don't Rate** AlertAction to **UIAlertActionStyle.default** This will basically make the AlertController look like the one in the Armchair screenshots. Is there a reason it's not like this by default? Thank you!
adam closed this issue 2025-12-29 18:28:49 +01:00
Author
Owner

@ArturoLee commented on GitHub (Aug 16, 2017):

Let me know if you would like me to do a pull for this. Thanks!

@ArturoLee commented on GitHub (Aug 16, 2017): Let me know if you would like me to do a pull for this. Thanks!
Author
Owner

@coneybeare commented on GitHub (Aug 18, 2017):

Hm, not sure why this got changed. Sure thing, please test and send a pull request for the change.

@coneybeare commented on GitHub (Aug 18, 2017): Hm, not sure why this got changed. Sure thing, please test and send a pull request for the change.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/Armchair#61