Closed ben-p-commits closed 1 year ago
linting issues noted. will address.
Let's just wait for the review of @thecoolwinter
@austincondiff , @lukepistrol - we should be all set to merge according to @thecoolwinter
I have the follow-on changes ready for CodeEdit.
I'll tag a new version in a couple of hours!
I just tagged the new release with 0.3.0
Background
This addresses bug #84 , and fulfills some of the requirements of #66 (the rest of which will be implemented in CodeEdit in a nascent PR).
Technical Details
Two things to note:
This will likely regress highlighting behavior- more detailed discussion available on this discord channel, but TLDR: @thecoolwinter has plans to obviate the need for the removed
NSLayoutManager
code when he migrates that work to TextKit 2.I went ahead and removed the invocation of STTextView's static scrollableTextView() helper function. It sets a number of properties which are immediately overwritten by our code in
loadView()
. It only obfuscates what's really happening here.