Preserve cursor more accurately

[sindresorhus]

Use the Autoprefixer source map to map the cursor back to its original position.

---

IssueHunt Summary

jdiogopeixoto has been rewarded.

Backers (Total: $40.00)

• issuehunt ($40.00)

Submitted pull Requests

• #72 Remove cursor position handling

---

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

---

IssueHunt has been backed by the following sponsors. Become a sponsor

[IssueHuntBot]

@IssueHunt has funded $40.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[jdiogopeixoto]

@sindresorhus I was looking into this and I removed the following line to check the default behaviour on Atom (v1.41.0).

atom-autoprefixer/index.js

Line 43 in ac699f1

editor.setCursorBufferPosition(cursorPosition);

What happens can be seen here. The cursor is on line 11 before saving and on line 13 after saving, remaining on the line with display: flex.

I think this is the correct behaviour because if I am writing in the file and saving it while writing, keeping the cursor in the same line that I was typing is helpful. Does this make sense?

[sindresorhus]

I guess Atom improved their diffing too. This was not as smooth before and the caret would jump around.

[issuehunt-oss]

@sindresorhus has rewarded $36.00 to @jdiogopeixoto. See it on IssueHunt

• 💰 Total deposit: $40.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $4.00