This pull request seeks to refactor the Inserter component to separate handling of insertion point and visibility. Previously the Inserter component was responsible for: setting the insertion point, showing the insertion point, and inserting a block at the current insertion point. To achieve this, it received two props insertIndex and insertionPoint which often -- but not always -- held the same value.

The changes here distinguish setting an insertion point to be the responsibility of where it is relevant; namely, the sibling inserter component. showInsertionPoint( index ) and hideInsertionPoint() have been replaced with toggleInsertionPointVisible( isVisible ) and setInsertionPointIndex( index ).

A hope here is to try to clarify a difference between an explicit insertion point (reflected in state.blockInsertionPoint) and the implicit insertion point (the logic contained within the getBlockInsertionPoint selector, inserting after content or the current block selection).

The main downside is that since there are now two separate actions for setting the insertion point index and its visibility, there may be separate renders which occur. This can be seen when, while a block is selected, you use the header inserter and then cancel both insertion and block selection by clicking on the editor background. The insertion point shows briefly at the bottom of the post content.

Testing instructions:

Verify that there are no regressions in the behavior of the insertion point, notably around selection (including multi-selection), sibling block insertion, and the header inserter (with and without selected block, in Visual and Text modes).

Ensure that unit tests pass:

npm test