The collection that the object is in can also have its selectability disabled, which is not being taken into account here. What this should do is call BKE_view_layer_base_find() and then check the selectable flag on the base. It could be wrapped in a utility function to avoid code duplication.

For the gizmo/design side I defer to @Campbell Barton (campbellbarton) though.