Skip to content

Commit efb7fa3

Browse files
author
Vladimir Enchev
committed
Time picker max hour fixed
1 parent 95f7061 commit efb7fa3

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

apps/tests/ui/time-picker/time-picker-tests.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,11 +76,11 @@ export function test_WhenCreated_MinHourIs1() {
7676
});
7777
}
7878

79-
export function test_WhenCreated_MaxHourIs23() {
79+
export function test_WhenCreated_MaxHourIs24() {
8080
helper.buildUIAndRunTest(_createTimePicker(), function (views: Array<viewModule.View>) {
8181
var timePicker = <timePickerModule.TimePicker>views[0];
8282
var actualValue = timePicker.maxHour;
83-
var expectedValue = 23;
83+
var expectedValue = 24;
8484
TKUnit.assert(actualValue === expectedValue, "Actual: " + actualValue + "; Expected: " + expectedValue);
8585
});
8686
}

ui/time-picker/time-picker-common.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import view = require("ui/core/view");
55
import types = require("utils/types");
66

77
function isHourValid(value: number): boolean {
8-
return types.isNumber(value) && value >= 1 && value <= 23;
8+
return types.isNumber(value) && value >= 1 && value <= 24;
99
}
1010

1111
function isMinuteValid(value: number): boolean {
@@ -48,7 +48,7 @@ export class TimePicker extends view.View implements definition.TimePicker {
4848
new proxy.PropertyMetadata(1, dependencyObservable.PropertyMetadataSettings.None, undefined, isHourValid));
4949

5050
public static maxHourProperty = new dependencyObservable.Property("maxHour", "TimePicker",
51-
new proxy.PropertyMetadata(23, dependencyObservable.PropertyMetadataSettings.None, undefined, isHourValid));
51+
new proxy.PropertyMetadata(24, dependencyObservable.PropertyMetadataSettings.None, undefined, isHourValid));
5252

5353
public static minuteProperty = new dependencyObservable.Property("minute", "TimePicker",
5454
new proxy.PropertyMetadata(undefined, dependencyObservable.PropertyMetadataSettings.None, undefined, isMinuteValid));

0 commit comments

Comments
 (0)