PowerToys icon indicating copy to clipboard operation
PowerToys copied to clipboard

[PTRun]Improve copy error handling

Open PesBandi opened this issue 4 months ago • 0 comments

Summary of the Pull Request

Improves how errors are handled when a copy operation fails. Gives the user more information and logs copy errors for all plugins.

PR Checklist

  • [x] Closes: #35412
  • [x] Communication: I've discussed this with core contributors already.
  • [x] Tests: All pass
  • [x] Localization: All end user facing strings can be localized
  • [x] Dev docs: No need
  • [x] New binaries: None
  • [x] Documentation updated: No need

Detailed Description of the Pull Request / Additional comments

  • Logs all copy errors (wasn't in ValueGenerator, Calculator, UnitConverter)
  • Displays a MessageBox when copying fails (wasn't in TimeDate)
  • Displays the exception message in the MessageBox

Validation Steps Performed

Observing that MessageBoxes appear when copying fails. I haven't tested the logger yet.

PesBandi avatar Oct 16 '24 19:10 PesBandi