-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Correct typo, CTL -> CTRL #7304
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Docs Build status updates of commit f6eecd0: ✅ Validation status: passedFor more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
|
@adegeo Can you check this one? |
|
The error isn't a false positive. These have no projects. The code needs to be upgraded. |
|
Oh, whoops. I thought you were looking for why the snippets build was failing. 😁 It's just updating some code comments using CTRL to identify the keyboard key. @gewarren @BillWagner Are we currently requiring minor updates like this to be recreated with full code examples? |
|
If I need to link these to some sort of project, I can give it a shot. I am not too familiar with c++ though. If there's any documentation regarding making said projects, send it my way and I can see if it's within my ability. |
|
I think we can let this change go without adding a project file since it's literally just updating code comments. |
|
Although now that I look at the code, I'm not sure that |
I loaded up the c# sample and am not seeing any issue stemming from the usage of e.KeyState, atleast on net framework 4.8. No plural form being made known to me by VS. |
|
@gewarren That API is the WPF API, this is the WinForms API which is probably why it's named slightly different. 😁 |
Summary
Corrects references to the CTRL key as CTL in System.Windows.Forms.Control.DoDragDrop
I was not able to find any information that would suggest this is correct.