Skip to content

Implement Settings screen, fix shiftTab, enable tab indent#234

Merged
sehugg merged 5 commits into
sehugg:masterfrom
fredsa:indent
Mar 24, 2026
Merged

Implement Settings screen, fix shiftTab, enable tab indent#234
sehugg merged 5 commits into
sehugg:masterfrom
fredsa:indent

Conversation

@fredsa

@fredsa fredsa commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

New settings menu
- Tab size (default 8)
- Insert spaces when pressing TAB
- Highlight special characters
- Highlight whitespace
- Highlight unwanted trailing whitespace
- Automatically add and remove closing brackets

@fredsa

fredsa commented Mar 9, 2026

Copy link
Copy Markdown
Contributor Author

(Partially) addresses #229

@fredsa fredsa force-pushed the indent branch 2 times, most recently from 7ac5306 to 5298811 Compare March 9, 2026 05:47
@fredsa fredsa force-pushed the indent branch 3 times, most recently from fd2b555 to f14bd57 Compare March 22, 2026 20:12
@fredsa

fredsa commented Mar 22, 2026

Copy link
Copy Markdown
Contributor Author

Here's what the settings screen looks like:

image

fredsa added 4 commits March 23, 2026 18:33
- Tab size (default 8)
- Insert spaces when pressing TAB
- Highlight special characters
- Highlight whitespace
- Highlight unwanted trailing whitespace
- Automatically add and remove closing brackets
shiftTab had incorrectly assumed all indentation was done with spaces
@sehugg sehugg merged commit 6cca8e8 into sehugg:master Mar 24, 2026
1 check passed
@sehugg

sehugg commented Mar 24, 2026

Copy link
Copy Markdown
Owner

I got smart indent behavior working on the main branch based on a assembler/not-assembler test, seems to work for me or at least similar to CM5.

I'm not sure how insert spaces = off (use tabs) interacts with smart indent, maybe smart indent is just disabled in this case since it always inserts spaces. (Or maybe it's fine since this is how CM5 worked)

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