Page MenuHomePhabricator

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

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



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

@ref T7901

Diff Detail

rEFL core/efl
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
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:

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 -

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.