Setfocus Problem

C

Craig M

Hi there....
I am trying to clear and setfocus to textbox1 after I enter a number!
I must be setting the controls wrong because I can't get textbox1 to accept
focus without hitting enter twice.
Here is the sample code below... please help!

Option Explicit

Private Sub TextBox1_AfterUpdate()
Dim varOne As Long
Dim varTwo As Long
Dim varTotal As Long
If TextBox1.Value <> "" And TextBox1 <> 0 Then
varOne = TextBox1.Value
varTwo = TextBox2.Value
varTotal = varOne + varTwo
TextBox2.Value = Format(varTotal, "Standard")
TextBox1.Value = ""
TextBox1.SetFocus
Else
TextBox1.Value = ""
TextBox1.SetFocus
End If
End Sub
Private Sub UserForm_Activate()
TextBox1.Value = ""
TextBox2.Value = Format(0, "Standard")
End Sub


Thanks
Craig
 
J

Jay

Hi Craig -

Your code looks good. Only problem is that the Exit event fires after your
AfterUpdate event procedure and it transfers focus to TextBox2, superceeding
your TextBox1.Setfocus statement.

Try adding an Exit event procedure. If that works for you, you can remove a
few unnecessary lines from your afterupdate event.

Jay
----------
Option Explicit

Private Sub TextBox1_AfterUpdate()

Dim varOne As Long
Dim varTwo As Long
Dim varTotal As Long
If TextBox1.Value <> "" And TextBox1 <> 0 Then
varOne = TextBox1.Value
varTwo = TextBox2.Value
varTotal = varOne + varTwo
TextBox2.Value = Format(varTotal, "Standard")
' TextBox1.Value = ""
' TextBox1.SetFocus
' Else
' TextBox1.Value = ""
' TextBox1.SetFocus
End If
End Sub

Private Sub TextBox2_Enter()
TextBox1.Value = ""
TextBox1.SetFocus
End Sub

Private Sub UserForm_Activate()
TextBox1.Value = ""
TextBox2.Value = Format(0, "Standard")
End Sub
 
S

Sharad

Hi Craig

The userform should be modeless for setfocus to work.
When you show the form add vbModeless at the end.
e.g.: Userform1.Show vbModeless

Sharad
 
J

Jay

Sorry, my previous post should read:

"Try adding an Enter event procedure (for TextBox2)." instead of
"Try adding an Exit event procedure."
 
C

Craig M

Thank You... adding the enter event for textbox2 did the trick!
Something so easy was killing me!

Thanks Again
Craig
 
T

Tom Ogilvy

Actually Jay was right the first time, but doesn't seem to know it.

Private Sub TextBox1_Exit(ByVal Cancel As MSForms.ReturnBoolean)
Dim varOne As Long
Dim varTwo As Long
Dim varTotal As Long
If TextBox1.Value <> "" And TextBox1 <> 0 Then
varOne = TextBox1.Value
varTwo = TextBox2.Value
varTotal = varOne + varTwo
TextBox2.Value = Format(varTotal, "Standard")
cancel = True
Else
TextBox1.Value = ""
cancel = True
End If
End Sub

Private Sub UserForm_Activate()
TextBox1.Value = ""
TextBox2.Value = Format(0, "Standard")
End Sub

So you don't need a kludge solution with setfocus and extra events. It is
usually best not to fight the behavior of the controls if you can help it.
 
C

Craig M

Tom, I tried your method. It worked after I placed a Textbox1.Value =""
above the first cancel=True.
The only problem is that it seems to create an endless loop, when I placed a
commandbutton to exit the form I am unable to use it.

Craig
 
D

Dave Peterson

One easy fix is to select that commandbutton to exit and change the
..takefocusonclick property to false.

But it looks like your code is written to never let you leave that text box. If
that's correct, you may want to change that other textbox to a label--just to
display the value.
 
T

Tom Ogilvy

Your just too restrictive. This will allow the user to exit if the textbox1
is blank (which is should be when they want to exit).

Private Sub TextBox1_Exit(ByVal Cancel As MSForms.ReturnBoolean)
Dim varOne As Long
Dim varTwo As Long
Dim varTotal As Long
If TextBox1.Value <> "" And TextBox1 <> 0 Then
varOne = TextBox1.Value
varTwo = TextBox2.Value
varTotal = varOne + varTwo
TextBox2.Value = Format(varTotal, "Standard")
TextBox1.Value = ""
Cancel = True
Else
If TextBox1.Value = "0" Then
TextBox1.Value = ""
Cancel = True
End If
End If
End Sub
 
C

Craig M

Thanks Tom... that did the trick!
Craig


Tom Ogilvy said:
Your just too restrictive. This will allow the user to exit if the
textbox1 is blank (which is should be when they want to exit).

Private Sub TextBox1_Exit(ByVal Cancel As MSForms.ReturnBoolean)
Dim varOne As Long
Dim varTwo As Long
Dim varTotal As Long
If TextBox1.Value <> "" And TextBox1 <> 0 Then
varOne = TextBox1.Value
varTwo = TextBox2.Value
varTotal = varOne + varTwo
TextBox2.Value = Format(varTotal, "Standard")
TextBox1.Value = ""
Cancel = True
Else
If TextBox1.Value = "0" Then
TextBox1.Value = ""
Cancel = True
End If
End If
End Sub
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Top