One function for multiple Textbox WPF

Question

I have 5 textboxes that gets editable on double click.

Below is method i have written for one textbox.

private void TextBox_MouseDoubleClick(object sender, MouseButtonEventArgs e)
        {
            txtFirstLctrTime.IsReadOnly = false;
            txtFirstLctrTime.Background = new SolidColorBrush(Colors.White);
            txtFirstLctrTime.Foreground = new SolidColorBrush(Colors.Black);
        }

Is there any way i can use same method for all text box instead of writing different method for all?? I am fairly new to programming

Just a design question: Why? Does the user expect to have to double-click a textbox to be able to edit? Usually the user knows: textbox gray => disabled, textbox white => editable. Why confuse the user? — Thorsten Dittmar
– Thorsten Dittmar, Commented Mar 31, 2015 at 12:42

nvoigt · Accepted Answer · 2015-03-31 12:39:57Z

3

You can atach this handler to all textboxes. Then you check the sender, because that's the textbox you actually clicked:

    private void TextBox_MouseDoubleClick(object sender, MouseButtonEventArgs e)
    {
        var textBox = sender as TextBox;

        textBox.IsReadOnly = false;
        textBox.Background = new SolidColorBrush(Colors.White);
        textBox.Foreground = new SolidColorBrush(Colors.Black);
    }

You should look into MVVM and data binding thought, having click-handlers and code-behind has it's limits.

answered Mar 31, 2015 at 12:39

nvoigt

77.8k28 gold badges103 silver badges156 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Richa Over a year ago

Thanks a lot. Worked perfectly.

Thorsten Dittmar Over a year ago

Please note that this may throw a NullReferenceException in case you attach the event handler to anything but a textbox. Shouldn't be a problem, just to point it out.

Richa Over a year ago

@ThorstenDittmar I have just attached it to textbox, so i guess there should not be any issue :)

Sajeetharan · Accepted Answer · 2015-03-31 12:40:38Z

1

Attach same handler to all textboxes and use sender argument to get textbox instance which raised event:

private void  MouseDoubleClick(object sender, MouseButtonEventArgs e)
{
     TextBox textBox = (TextBox)sender;
    textBox.IsReadOnly = false;
    textBox.Background = new SolidColorBrush(Colors.White);
    textBox.Foreground = new SolidColorBrush(Colors.Black);
}

answered Mar 31, 2015 at 12:40

Sajeetharan

223k65 gold badges370 silver badges409 bronze badges

Comments

MajkeloDev · Accepted Answer · 2015-03-31 12:43:01Z

0

Yes, there is a way. Sender is a parameter which can tell You which controll fired this event. Look at my example below:

private void TextBox_MouseDoubleClick(object sender, MouseButtonEventArgs e)
{
  TextBox tbWhichFiredThisEvent = sender as TextBox;
  if(tbWhichFiredThisEvent != null)
  {
    tbWhichFiredThisEvent.IsReadOnly = false;
    // ... etc.
  }
}

answered Mar 31, 2015 at 12:43

MajkeloDev

1,66115 silver badges30 bronze badges

Comments

Ignacio Soler Garcia · Accepted Answer · 2015-03-31 12:57:32Z

0

Another option is to inherit from TextBox and implement your specific behavior on the OnDoubleClick method.

This way you can have this control on different views without repeated code.

answered Mar 31, 2015 at 12:57

Ignacio Soler Garcia

22k34 gold badges140 silver badges218 bronze badges

2 Comments

Richa Over a year ago

Any way i can select combobox only in particular panel and use above method?

Ignacio Soler Garcia Over a year ago

Sorry, I don't get what you mean

Collectives™ on Stack Overflow

One function for multiple Textbox WPF

4 Answers 4

3 Comments

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

3 Comments

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related