Comments on: Yet Another TimePicker Update – DisplaySeconds

By: Michael Bell

Michael Bell — Mon, 24 May 2010 19:30:56 +0000

I have implemented your fix and placed it here and on CodePlex.

chris :
Hi,
relating to the TwentyFour bug – 09:00 always getting 21:00, 10:00 always
getting 22:00 and so on…
don’t know if that’s really a good solution, but it worked for me:
IMHO there was just a little piece of code missing in the date property:
///
/// DateTime value. Defaults to current day if not set. Otherwise, maintains a value of the date assigned plus time shown.
///
[Browsable(true)]
public DateTime Date
{
get
{
if (ViewState["Date"] == null)
return DateTime.MinValue;
else
{
if (SelectedTimeFormat == TimeFormat.Twelve)
{
ViewState["Date"] = Convert.ToDateTime(ViewState["Date"]).ToShortDateString() + ” ” + Hour.ToString() + “:” + Minute.ToString() + “:” + Second.ToString() + ” ” + AmPm.ToString();
}
else
{
ViewState["Date"] = Convert.ToDateTime((Convert.ToDateTime(ViewState["Date"]).ToShortDateString() + ” ” + Hour.ToString() + “:” + Minute.ToString() + “:” + Second.ToString() + ” “), System.Globalization.CultureInfo.CurrentCulture.DateTimeFormat);
}
return Convert.ToDateTime(ViewState["Date"]);
}
}
set
{
ViewState["Date"] = value;
SetTime(value);
}
}
maybe someone can benefit from

By: Michael Bell

Michael Bell — Mon, 24 May 2010 19:21:02 +0000

@Jean I have implemented your fix. FOcus issues are hopefully resolved when using more than one instance in a page!

By: Terry

Terry — Wed, 07 Apr 2010 09:56:01 +0000

How to set all zero for hour, minute and second? Example: 00:00:00

I am using SelectedTimeFormat=”TwentyFour”.

Thanks.

By: Mark

Mark — Tue, 30 Mar 2010 16:14:32 +0000

From what I found and seems to have fixed it for me, is the fact that if the control is in ‘24′ hour mode, the ampm attribute is not created, therefore when LoadPostData() executes, since it’s not found, it defaults to PM. I added the following change (locate ’string sAmPm’ in the LoadPostData() function) and has been working fine so far.

// If missing, should be in 24 hour
string sAmPm = values[this.UniqueID + "_txtAmPm"];
if (!String.IsNullOrEmpty(sAmPm))
{
AmPmSpec tAmPm = (values[this.UniqueID + "_txtAmPm"] == “AM”) ? AmPmSpec.AM : AmPmSpec.PM;
if (tAmPm != AmPm)
{
AmPm = tAmPm;
hasAmPmChange = true;
}
}
else
{
if (int.Parse(sHour) > 11)
{
AmPm = AmPmSpec.PM;
}
else
{
AmPm = AmPmSpec.AM;
}
}

By: Louie

Louie — Wed, 24 Mar 2010 02:49:41 +0000

No more support for this timepicker? I also have the twentyfour hour problem, it alsways display 10am as 22:00. I’ll just create my own and post it later.

By: cdw

cdw — Tue, 09 Feb 2010 01:40:24 +0000

Has anyone successfully built a patched version that corrects the 24 Hour problem? Would you like to make the source and/or DLL available?

By: Jessy

Jessy — Mon, 08 Feb 2010 22:26:30 +0000

@chris I tied adding this to my solution but it's still not working. Can you please provide step by step instructions on where and how to add his code? Thank you SO much in advance!

By: chris

chris — Mon, 01 Feb 2010 20:52:18 +0000

Hi,
relating to the TwentyFour bug – 09:00 always getting 21:00, 10:00 always
getting 22:00 and so on…
don’t know if that’s really a good solution, but it worked for me:

IMHO there was just a little piece of code missing in the date property:

///
/// DateTime value. Defaults to current day if not set. Otherwise, maintains a value of the date assigned plus time shown.
///
[Browsable(true)]
public DateTime Date
{
get
{
if (ViewState["Date"] == null)
return DateTime.MinValue;
else
{
if (SelectedTimeFormat == TimeFormat.Twelve)
{
ViewState["Date"] = Convert.ToDateTime(ViewState["Date"]).ToShortDateString() + ” ” + Hour.ToString() + “:” + Minute.ToString() + “:” + Second.ToString() + ” ” + AmPm.ToString();
}
else
{
ViewState["Date"] = Convert.ToDateTime((Convert.ToDateTime(ViewState["Date"]).ToShortDateString() + ” ” + Hour.ToString() + “:” + Minute.ToString() + “:” + Second.ToString() + ” “), System.Globalization.CultureInfo.CurrentCulture.DateTimeFormat);
}
return Convert.ToDateTime(ViewState["Date"]);
}
}
set
{
ViewState["Date"] = value;
SetTime(value);
}
}

maybe someone can benefit from

By: Frank

Frank — Wed, 20 Jan 2010 21:19:40 +0000

Thank you Jean for posting your solution.
works great!

By: Patrick

Patrick — Fri, 15 Jan 2010 14:29:32 +0000

@Ilia Lukianov Hi Ilia, could u please send the source code for fix the SelectedTimeFormat=”TwentyFour”?