Buttons Enabled Property not Working Properly

Question

Hi all,

I am creating an Windows Application. I have two buttons. I have written the following code snippet.

frmRb obj = new frmrb();
private void btnPd_Click(object sender, EventArgs e)
        {
           btnCancel.Enabled = true;
           obj.btnRtn.Enabled = true;
        }

        private void btnCancel_Click(object sender, EventArgs e)
        {
            this.Close();
            obj.BringToFront();
            obj.Focus();
        }

The above coding does not generate any error.

All the statements are working properly but the following statement is not working properly:

obj.btnRtn.Enabled = true;

is not executed.

The frmrb forms is bring to front and it is focussed but btnRtn is not Enabled that is the statement obj.btnRtn.Enabled = true; is not working.

By default I have set the property of btnRtn Enabled to false. And Please note the Modifier property of btnRtn button is set to PUBLIC.

Now how should I change the coding so that I can get this statement executed.

obj.btnRtn.Enabled = true;

Can anybody help me out?

Thanks in Advance!!

Answer 1

I strongly suspect that either the click handler isn't being called or you're not looking at the form you think you are.

Where are you setting up the click handler for btnPd_Click? Check that's still wired up correctly.

Then put a breakpoint on the first line of the handler and run it in the debugger - if you don't hit the breakpoint when you click the button, that's the problem.

EDIT: Okay, next steps:

• Check that you're looking at the right button. Change its text as well as enabling it.
• Check that its container is enabled, as suggested by phoenix. Not just its direct parent, but all the way up.
• Check what else your UI thread is doing afterwards - are you blocking it for some reason? Does the rest of the UI work at that point?

Answer 2

I've been using VB rather than C#, but the languages are very similiar. in VB, you have to add "handles SomeButton.Click" to make it handle the Click events.

According to google, the equivalent in C# is to go into the Designer.cs file, find where the controls are, and manually change the click event hookup to point to your new event handler.

As mentioned by the previous poster, use a breakpoint (F9) and the debugger to see if that method is ever called when you execute the event. If it is not called, then the problem is probably not with the enabled property, but the wiring of the method so that it is invoked when the event occurs.

Here's a reference:

http://www.tek-tips.com/viewthread.cfm?qid=1442702&page=5

Answer 3

Is the button placed inside a panel or any container. If yes then please check the enabled status of the container also.

Answer 4

Are you sure the obj is the form that is bought to the front? Can you post a sample app with the code not working?

Answer 5

Just a hunch. May be the 'Locked' property of button is 'true'

Answer 6

You not mention that where you obj(which is the instance of frmRb) show. because it is very important point. from your coding it seem that frmRb is already visible. so u never called the

obj.Show() ;

instead you call the

obj.BringToFront();

so the problem is that you never show the frmRb object. which is you create u in 1st line. each time u write the line

frmRb obj = new frmrb();

new instance of frmrb is created. So u must again show it, with the line obj.Show() ; Now u rewrite ur code as ::

frmRb obj = new frmrb();
private void btnPd_Click(object sender, EventArgs e)
        {
           btnCancel.Enabled = true;
           obj.btnRtn.Enabled = true;
        }

        private void btnCancel_Click(object sender, EventArgs e)
        {
            this.Close();
            obj.Show();
            obj.BringToFront();
            obj.Focus();                
        }

I hope it is helpful for u and solve ur problem.

Answer 7

I would try just switching the sequence of statements to: private void btnPd_Click(object sender, EventArgs e) { obj.btnRtn.Enabled = true; btnCancel.Enabled = true; }

and see if that helps you debug

Answer 8

Your code really should produce an error... C# is case sensitive, meaning frmRb is not the same as frmrb. Anyway, I copied it, created 2 forms and 3 buttons, and set up the handler, and it worked fine.

private void InitializeComponent()
{
    this.btnPd = new System.Windows.Forms.Button();
    this.btnPd.Location = new System.Drawing.Point(90, 116);
    this.btnPd.Name = "btnPd";
    this.btnPd.Size = new System.Drawing.Size(75, 23);
    this.btnPd.TabIndex = 1;
    this.btnPd.Text = "button1";
    this.btnPd.UseVisualStyleBackColor = true;
    this.btnPd.Click += new System.EventHandler(this.btnPd_Click);
}
public System.Windows.Forms.Button btnRtn;

Are you sure you were handling btnPd? Perhaps you may have locked your enabling code inside a disabled button? Hopefully this small working sample helps you find the problem. As for the rest of the code, All I changed was the frmRb to frmrb so they match.

Answer 9

If the form that your are trying to show on the screen is properly shown, may be you can try this:

1. Create a public method in the form that will set the button property enabled = true;
2. After creating the form, and showing, you can call that public method;

frmRb obj = new frmrb(); obj.EnableButton