Skip to content
This repository was archived by the owner on Oct 23, 2019. It is now read-only.

minutesStep, secondsStep and showSeconds options#7

Open
korun wants to merge 8 commits into
add-timepickerfrom
time-options
Open

minutesStep, secondsStep and showSeconds options#7
korun wants to merge 8 commits into
add-timepickerfrom
time-options

Conversation

@korun

@korun korun commented Dec 21, 2014

Copy link
Copy Markdown
Collaborator

Added minutesStep, secondsStep and showSeconds options.
But there is a problem:

Test "Options: showSeconds" assertion failed. Expected <2012-03-05 15:09:02.00> Actual <2012-03-05 15:00:02.00>

If we don't touch minutes select box - it's not set thyself. O_o

@rchavik, what do you think, what is it?

@korun

korun commented Dec 21, 2014

Copy link
Copy Markdown
Collaborator Author

BTW, if I run tests/tests.html in the browser - the test is passed!

Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:34.0) Gecko/20100101 Firefox/34.0

@rchavik

rchavik commented Dec 22, 2014

Copy link
Copy Markdown
Owner

Uhhh.. not sure. wild guess, maybe something wrong in _[get|set]TimeDropdowns()

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants