Merged
Conversation
…o mobile_stepper_styling
Co-Authored-by: Kin Chong wongkinchong75@gmail.com
Co-Authored-by: Kin Chong wongkinchong75@gmail.com
Contributor
There was a problem hiding this comment.
Code Review
This pull request introduces breakpoint navigation to the substitution visualizer, allowing users to jump between debugger statements via updated UI buttons and hotkeys. The implementation includes new logic to scan content markers for debugger statements and adds a renderer for them. Feedback focuses on improving maintainability by removing commented-out code, refactoring duplicated SCSS styles into reusable mixins or placeholders, and replacing hardcoded color values with SASS variables.
10 tasks
martin-henz
requested changes
Apr 2, 2026
Member
martin-henz
left a comment
There was a problem hiding this comment.
Please address the issues pointed out by gemini-bot.
Pull Request Test Coverage Report for Build 23936888671Details
💛 - Coveralls |
martin-henz
requested changes
Apr 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR depends on the js-slang PR [https://github.com/source-academy/js-slang/pull/1926]. The frontend build will fail until the js-slang changes are merged and published to npm, and package.json is updated to the new version."
This PR allows users to access the step with the next/previous breakpoint in the stepper
Type of change
How to test
Checklist