Thanks Roger

I substituted the original with your suggestion but it still doesn't work.

I have the cells formatted as custom h:mm.

When an input in excess of 2359 is made the message 'You did not enter a
valid time' is given.

In the formula bar it shows '12/02/1907 00:00:00' when 2600 is entered for
instance.

As you probably appreciate VBA is not my forte and I am grateful for any
assistance.

Regards

--
Russell Dawson
Excel Student

Please hit "Yes" if this post was helpful.


"Roger Govier" wrote:

Hi Russell

Case 4 ' e.g., 1234 = 12:34
If .Value 2359 Then GoTo EndMacro

TimeStr = Left(.Value, 2) & ":" & _
Right(.Value, 2)

--
Regards
Roger Govier

Russell Dawson wrote:
I have the following code (courtesy of this discussion group) in a w/b that
allows time taken on a task to be input and calculated.
My problem now is that it does allow for input of hours worked in excess of
23:59.
Is anybody able to suggest how the code can be amended to allow for this.
Code is:
Private Sub Worksheet_Change(ByVal Target As Excel.Range)
Dim TimeStr As String

On Error GoTo EndMacro
If Application.Intersect(Target, Union(Range("E:E"), Range("g:g"),
Range("i:i"), Range("k:k"), Range("m:m"), Range("o:o"), Range("q:q"),
Range("s:s"), Range("u:u"), _
Range("w:w"), Range("y:y"))) Is Nothing Then
Exit Sub
End If
If Target.Cells.Count 1 Then
Exit Sub
End If
If Target.Value = "" Then
Exit Sub
End If

Application.EnableEvents = False
With Target
If .HasFormula = False Then
Select Case Len(.Value)
Case 1 ' e.g., 1 = 00:01 AM
TimeStr = "00:0" & .Value
Case 2 ' e.g., 12 = 00:12 AM
TimeStr = "00:" & .Value
Case 3 ' e.g., 735 = 7:35 AM
TimeStr = Left(.Value, 1) & ":" & _
Right(.Value, 2)
Case 4 ' e.g., 1234 = 12:34
TimeStr = Left(.Value, 2) & ":" & _
Right(.Value, 2)
Case 5 ' e.g., 12345 = 1:23:45 NOT 12:03:45
TimeStr = Left(.Value, 1) & ":" & _
Mid(.Value, 2, 2) & ":" & Right(.Value, 2)
Case 6 ' e.g., 123456 = 12:34:56
TimeStr = Left(.Value, 2) & ":" & _
Mid(.Value, 3, 2) & ":" & Right(.Value, 2)
Case Else
Err.Raise 0
End Select
.Value = TimeValue(TimeStr)
End If
End With
Application.EnableEvents = True
Exit Sub
EndMacro:
MsgBox "You did not enter a valid time"
Application.EnableEvents = True
End Sub







.