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 small backwards-compatible features to language, improve editing experience #333

Merged
merged 12 commits into from
Feb 11, 2025

Conversation

cyderize
Copy link
Collaborator

@cyderize cyderize commented Feb 6, 2025

These changes are deployed here.

Bug fixes:

  • Fix a bug where long output in the output window was unable to be scrolled all the way to the top

Language changes:

  • Change the 0xA opcode to LoadImmi X, which loads a 12-bit unsigned immediate value into the AC.
  • Change Clear to be an alias for LoadImmi 0
  • Allow using 0C00 as the operand for SkipCond to allow for skipping when the AC is non-zero.

Editor features:

  • Add an instruction set button to the toolbar which shows help for syntax and instructions
  • Enable using Ctrl+/ to toggle line comments
  • Allow auto-complete to appear in more situations with proper context sensitive suggestions
  • Show instruction and label details in auto-complete prompts
  • Show tooltip with details when hovering over instructions in the code
  • Make the user input prompt more visible when an Input instruction is executed
  • Improve text contrast in the status bar

Code changes:

  • Update code to use Svelte 5 Runes (was previously written for Svelte 4)

@cyderize cyderize requested a review from auroranil as a code owner February 6, 2025 05:06
auroranil
auroranil previously approved these changes Feb 10, 2025
@auroranil
Copy link
Member

Do we want to version this PR?

@cyderize
Copy link
Collaborator Author

Do we want to version this PR?

Yes, I think so. I forgot to do that - I'll fix that later today

@cyderize cyderize merged commit 392ceb1 into MARIE-js:main Feb 11, 2025
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.

2 participants