Change button text#312
Change button text#312david-poindexter merged 2 commits intonvisionative:developfrom EPTamminga:develop
Conversation
Change button text
Update ViewExistingSites.Designer.cs
valadas
left a comment
There was a problem hiding this comment.
While we are at it could we rename metroButton2 to btnClose or some such ? Other than that, I am good with this change either way.
@valadas I agree with the recommended change. Let's go ahead and merge this PR and we can create a new PR for the control name change. @EPTamminga I am also going to update your PR description so #311 will be auto-closed upon merging this PR. Thanks for your contribution! 🎉 |
|
@david-poindexter: it was a massive change :-) (While I made the issue, my idea was to solve it in the same round) |
|
Thanks @EPTamminga - no worries. A change to the control name would need to be built and tested locally anyway. So it was just as easy to split it up and get the cosmetics rolled in via this PR. |
Resolves #311