Skip to content

Add test for highlighter#33

Merged
ricklupton merged 1 commit intoricklupton:mainfrom
Scrybbling-together:highlighter
Apr 4, 2026
Merged

Add test for highlighter#33
ricklupton merged 1 commit intoricklupton:mainfrom
Scrybbling-together:highlighter

Conversation

@Azeirah
Copy link
Copy Markdown
Contributor

@Azeirah Azeirah commented Mar 21, 2025

This adds a .rm file which uses the Highlighter tool with color code = 9.

This shouldn't fail, but it does, even on rmc 0.3.0

@Azeirah
Copy link
Copy Markdown
Contributor Author

Azeirah commented Mar 21, 2025

This failure is caused by the fact that the highlighter tool is commented out in the colors dict

#! PenColor.HIGHLIGHT = ...

I think this was done because the highlighter's color is actually stored/computed elsewhere? This is related to ricklupton/rmscene#44, right @ChenghaoMou?

@ChenghaoMou
Copy link
Copy Markdown
Contributor

ChenghaoMou commented Mar 21, 2025

Yes, it is related. #34 should fix this issue once the new version of rmscene is integrated.

@ricklupton ricklupton merged commit ecf80ba into ricklupton:main Apr 4, 2026
0 of 6 checks passed
@ricklupton
Copy link
Copy Markdown
Owner

I know it's been a while, but thanks for this.

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.

3 participants