correctly show minutes in day view

[ghost]

No description provided.

[dougwo]

I temporarily fixed this as follows:

calendar.js line 534:

from
var m = "" + (time_split * part + (hour == 0) ? parseInt(time_start[1]) : 0);

to
var m = "" + (time_split * part);

I don't know what the last part was for, but it was confusing... This may have wider-reaching implications

[geskill]

the last part:

((hour == 0) ? parseInt(time_start[1]) : 0))

this is for setting the start time considering minutes, too. minutes ll be added to the first hours.

example:
start_time: 07:30
time_split: 30

then my calendar starts with: 07:30 and 08:00 is the next row.

[dougwo]

Well it doesn't work. Look at the demo page! It just repeats the hour and no minutes...

[geskill]

because brackets are missing ...

(hour == 0) ? parseInt(time_start[1]) : 0)

fix it to the following and it works ... therefore i said, this patch is needed (see here: #335)

((hour == 0) ? parseInt(time_start[1]) : 0))

[dougwo]

Ah. I missed that at first. thanks!

[ghost]

Any reason this hasn't been merged yet?

[josh-m-sharpe]

bump.

[Serhioromano]

Thank you.