fix: getNearestBlockContainerPos
unclear
#1241
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #1237
Currently,
getNearestBlockContainerPos
gets the nearest block, which may be acolumnList
,column
, orblockContainer
. I think the functionality is fine, but this PR renames it togetNearestBlockPos
for better clarity and removes the error ingetBlockInfoFromSelection
for when something other than ablockContainer
is returned. This fixes a bug that caused said error to be thrown by selecting acolumn
node using triple click (I think) or Cmd+click.