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

Develop #9

Merged
merged 4 commits into from
Nov 14, 2024
Merged

Develop #9

merged 4 commits into from
Nov 14, 2024

Conversation

tcorzo
Copy link
Owner

@tcorzo tcorzo commented Nov 14, 2024

This pull request includes several changes to improve the functionality and user experience of the register tables in the project. The most important changes include adding context menu support for register tables, updating the README.md to reflect the current state of the project, and modifying the .pre-commit-config.yaml file to include a new CI configuration.

Enhancements to register tables:

  • src/components/RegisterTable.vue: Added context menu support for register tables, including options to insert above, insert below, and delete registers. This involved significant changes to the script setup, including defining props, setting up the context menu, and handling context menu actions. [1] [2]
  • src/components/Editor.vue: Simplified the RegisterTable component usage by removing unnecessary editable attribute bindings.
  • src/components/emulator/AuxRegisters.vue and src/components/emulator/DataRegisters.vue: Removed the editable attribute from RegisterTable components as it is now handled internally. [1] [2]

Documentation updates:

  • README.md: Updated the project task list to reflect the current state of the project, including marking the register insert/delete task as completed and adding new tasks for auto-scrolling and register re-addressing.

Configuration changes:

  • .pre-commit-config.yaml: Added a new CI configuration to auto-update the branch to "develop" during pre-commit checks.

@tcorzo tcorzo self-assigned this Nov 14, 2024
@tcorzo tcorzo merged commit 18aead3 into master Nov 14, 2024
4 checks passed
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