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

Add C64 paste from clipboard functionality #112

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

highbyte
Copy link
Owner

@highbyte highbyte commented Sep 6, 2024

PR Classification

New feature to enable clipboard text pasting functionality in the C64 emulator.

PR Summary

Introduced clipboard text pasting across different UI components and updated relevant classes to support this feature.

  • Use NuGet packageTextCopy to read clipboard in native and Blazor WASM apps.
  • C64MenuConsole: Added "Paste" button and implemented event handler for pasting text.
  • C64Menu.razor: Included "Paste" button and implemented PasteText method.
  • C64: Added C64TextPaste property and refactored constructor for text-pasting logic.
  • C64Keyboard: Updated to return a boolean for character insertion.
  • Program.cs: Registered TextCopy service for clipboard operations.

Integrated the `TextCopy` library to enable clipboard operations across multiple projects. Added a "Paste" button to the `C64MenuConsole`, `SilkNetImGuiMenu`, and Blazor-based `C64Menu.razor` components to allow pasting text into the C64 emulator. Implemented methods to handle paste operations, adjusted UI layouts, and updated control states based on the emulator's state. Enhanced the `C64` class to map ANSI characters to PETSCII and insert them into the keyboard buffer. Updated project files to include the `TextCopy` package and added logging and error handling for clipboard operations.
Copy link

sonarqubecloud bot commented Sep 6, 2024

@highbyte highbyte merged commit 1cd8b08 into master Sep 6, 2024
5 checks passed
@highbyte highbyte deleted the feature/c64-paste-basic-text branch September 6, 2024 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant