For better understanding of the property, "is_24hour" looks
better than "ampm".
It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/
Looks good to me. It seems that 24 hour is more familiar at least in the android developers' world as well - https://developer.android.com/reference/android/widget/TimePicker.html#is24HourView()