Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Indent Works Properly When AutoIndent Is Unchecked #11677

Closed
wants to merge 6 commits into from

Conversation

ebroom2014
Copy link

This is in regard to #10407

Before when the AutoIndent was unchecked and you were to copy and paste a section of text with a newline character, it would still indent. See bug report for more details. Here is a video of the original bug.

In this changed code in selection.coffee it checks to see if AutoIndent is checked before it does the indent adjustment. You can see atom working properly here.

@ebroom2014
Copy link
Author

Also, I updated the Linux README because I had some troubleshooting errors that were not written about in the README and I know of others who have had the same problem. There were also some spelling errors that I fixed.

Also, I added an uninstall portion of the Linux README because I could not find a compiled list of how to uninstall Atom anywhere. I thought this would be helpful for others.

@winstliu
Copy link
Contributor

winstliu commented May 3, 2016

You seem to have inadvertently reverted many recent changes to the Linux build instructions. Can you please create a new PR for the build instruction changes to make reviewing easier? Thanks.

@ebroom2014
Copy link
Author

ebroom2014 commented May 3, 2016

I did inadvertently revert some changes to the Linux build instructions. I have pushed the correct file for PR up. Thank you for noticing this.

EDIT: I realize that the last push I did, you couldn't see the difference between my changes and the original document since I just updated my document to the original changes. So I pushed the current Linux README and then pushed my changes so you can see the side-by-side changes. View this by looking at the latest commit, 72a7ed4, below.

@sadick254
Copy link
Contributor

Hey @ebroom2014 , I know this PR has been open for a long time. Apologies for not being able to look into it in a timely manner. There have been many changes on Atom since this PR was authored i.e We migrated a good number of our source files from CoffeeScript to Javascript`. This has led to some conflicts on this PR that are not easily resolvable.

I have created a separate PR that supersedes this one. I will close this for now but don't hesitate to reach out in case of any queries.

Thank you for your contribution.

@sadick254 sadick254 closed this May 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants