Page MenuHomePhabricator

efl_ui_timepicker: change a property name (ampm -> is_24hour)
ClosedPublic

Authored by woohyun on Sep 1 2019, 11:07 PM.

Details

Summary

For better understanding of the property, "is_24hour" looks
better than "ampm".

@ref T7901

Diff Detail

Repository
rEFL core/efl
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13068
Build 9295: arc lint + arc unit
woohyun created this revision.Sep 1 2019, 11:07 PM

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/

woohyun requested review of this revision.Sep 1 2019, 11:07 PM
kimcinoo accepted this revision.Sep 1 2019, 11:38 PM
kimcinoo added a subscriber: kimcinoo.

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()

This revision is now accepted and ready to land.Sep 1 2019, 11:38 PM
segfaultxavi accepted this revision.Sep 2 2019, 12:09 AM
This revision was automatically updated to reflect the committed changes.