fix(scraps): align leadingItems in compactSelect with check box/icon

[TkDodo]

This is the second attempt to fix the alignment, after #106167 was reverted because it introduced an issue in the environment selector.

The problem was that, in order to do the alignment, we wrapped the leadingItems again in another container, because the internally created checkbox is also wrapped.

This fixed the alignment issue, however there are cases where hideCheck:true is passed to hide the internal checkbox and then a separate checkbox is rendered inside leadingItems 🤯 .

This is what HybridFilter does, and that checkbox should never get wrapped because of how pointer events are handled.

The approach in this PR always just plainly returns the leadingItems when hideCheck is passed to never do any wrapping. This is also what happened before, but it’s now a bit more explicit. This is the important change:

02cee3c

The rest are the exact changes re-applied from the reverted PR.

Note: I think HybridFilter only does this because it has an option to also show the checkbox on the right side, which is used by the Project selector. If we streamline this, like @Jesse-Box has planned, we can probably just use the internal checkbox from CompactSelect and remove all of these workarounds.