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

Implemented codemirror extension to keep initial indentation for wrapped lines #192

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Maran1947
Copy link
Contributor

Resolves #187

@heyman
Copy link
Owner

heyman commented Feb 11, 2024

#187 is about displaying an indentation for lines that wraps, so this is not really a solution for it.

@heyman
Copy link
Owner

heyman commented Feb 11, 2024

Though this may also be desirable, I need to think more about that.

@NoahAndrews
Copy link
Contributor

I thought that this was the original request as well. I think they're both desirable behaviors.

@Maran1947
Copy link
Contributor Author

Sorry, I was misunderstood the issue. So, It appears there are two separate scenarios regarding indentation to address:

  1. Upon hitting 'Enter', the new line fails to adopt the indentation of the preceding line.
  2. Fails to maintain proper indentation for lines that wrap or extend beyond the width of the viewport.

And the issue is about scenario 2, isn't?

@jacobcarpenter
Copy link

And the issue is about scenario 2, isn't it?

Correct. #187 is talking about the word-wrap column. If you look closely at the first screenshot in that issue, you'll see that the line number in the margin shows that line 3 wrapped (by leaving a gap in the line numbering for the next line before showing 4).

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.

Keep initial indentation for wrapped lines
4 participants