Rename dialog - select all text in textbox when loading

[ghost]

I admit that I don't understand how this works...

But eventually I will learn enough to understand.

Closes #3090

[rubberduck203]

So, what you've done here is provided an anonymous (unnamed) function that gets called by the base class when the view is loaded. (Otherwise known as a delegate or callback).

Is there a specific thing you don't understand?

[rubberduck203]

@retailcoder @Hosch250 my MVVM is rusty. Is there a way to accomplish this outside of the code behind?

[retailcoder]

@rubberduck203 haven't seen the code yet, but IMO that's strictly a presentation concern, it has nothing to do in the ViewModel - it actually belongs in the View :)

[retailcoder]

You could remove the delegate keyword here, C# is clever enough to infer it :-)

The problem with anonymous handlers is that when the event provider is destroyed, there's no way to de-register the handler; if the event provider wasn't the same object that registers the event, this would be a memory leak (the handler is holding on to a reference that's otherwise eligible for garbage collection. Doesn't apply in this case, but definitely worth understanding.

[ghost]

That line is where I was confused. Found it on the interwebs lol. IntellSense told me what base is, but no idea what delegate is or += does.

I understand everything in the second paragraph, but what an "anonymous" handler is. I'll probably get the answer to that this holiday weekend as I work though as much C# tutorials as I can handle. :)

Edit: I think I get it... maybe.

[Hosch250]

Not sure, but can't you do this in the XAML?

[ghost]

@Hosch250 I would have thought so... but I didn't find any method or property to do it. YMMV.

[Hosch250]

I don't think you can. But, this does need to be presentation-only. It should go in the code-behind (the .xaml.cs file).

[ghost]

It should go in the code-behind (the .xaml.cs file).

@Hosch250 Um... Isn't that where I put it? RetailCoder.VBE/UI/Refactorings/Rename/RenameView.xaml.cs

[Hosch250]

Yeah, I was just making a random comment. Remove the delegate keyword and it looks good to go.