Skip to content
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

IP-Scanner: Copy\MAC-Address - Copies Vendor instead of MAC-Address #2642

Closed
nhwCoder opened this issue Mar 14, 2024 · 2 comments · Fixed by #2644
Closed

IP-Scanner: Copy\MAC-Address - Copies Vendor instead of MAC-Address #2642

nhwCoder opened this issue Mar 14, 2024 · 2 comments · Fixed by #2644
Labels
Milestone

Comments

@nhwCoder
Copy link

nhwCoder commented Mar 14, 2024

When I use Copy\MAC-Address in IP-Scanner it copies "Vendor" instead of "MAC-Address" top clipboard.

Steps to reproduce the issue:

  1. Open IP-Scanner
  2. RightMouse-click any entry row
  3. Copy\MAC-Address
  4. Content in clipboard is "Vendor" and not "MAC-Address"

NETworkManager version: 2024.1.7.0 (= newest)

  • [x ] Portable

Windows version (like 10.0.x.x): 10.0.22631 Build 22631(= newest)

.NET version (like 6.0.x): 8.x

ADDITIONAL

  • In RightMouse-click menu there should be also an entry "Copy all" since Ctrl+C is already working!
@BornToBeRoot
Copy link
Owner

BornToBeRoot commented Mar 14, 2024

Thanks @nhwCoder for the report.

Fixed via #2644 for the next release!

I'm not sure if it makes sense to implement the copy all by myself if you Ctrl+C already works out of the box for the coloumns.

@mergify mergify bot closed this as completed in #2644 Mar 14, 2024
@BornToBeRoot BornToBeRoot added this to the next-release milestone Mar 14, 2024
@nhwCoder
Copy link
Author

I'm not sure if it makes sense to implement the copy all by myself if you Ctrl+C already works out of the box for the coloumns.
Not implementing by yourself, but adding it to the menu to see this feature & and in C# trigger a SendKey Ctrl+C would solve it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants