Use code instead of Macro

Me.[Vehiclessbf].Visible = (Me.[Account#]="392")

When the cursor is loacted in the AfterUpdate event of the Account field,
there is a button with three dots, press it and select code view.
Enter the above line in there

Also, if you are moving between records, add the above line to the MainForm
OnCurrent event

your macro tells Access when to *show* the subform, but does nothing to tell
Access when to *hide* the subform. try adding a couple additional actions to
your macro, as

First line of macro:
Macro Name: "Hide/ShowVehiclesForm";
Condition:  [Forms]![CapitalItems]![Account#]="392"
Action:  "SetProperty"
Control Name:  Vehiclessbf
Property:  Visible
Value:  Yes

Second line of macro:
Macro Name: <leave blank>
Condition:  ... <add the three dots, which are called an ellipsis>
Action:  StopMacro

Third line of macro:
Macro Name: <leave blank>
Condition:  <leave blank>
Action:  "SetProperty"
Control Name:  Vehiclessbf
Property:  Visible
Value:  No

with the above setup, the macro runs each time the control's AfterUpdate
event fires. when the condition on the first line is met, the subform is
unhidden and the macro stops on the second line. when the condition on the
first line is *not* met, the macro skips to the third line and hides the
subform.

as Ofer's post demonstrates, VBA code can do the same thing with a much
shorter statement. i encourage you to begin learning VBA, which is much more
powerful than macros (i'm guessing that still holds true even in A2007); but
don't feel you're wasting time working with macros. for one thing, they
teach you the importance of thinking through both (or all) "sides" of an
action when you're manipulating objects programmatically.

hth

want

392.