Change `logout_direct` uses to `gitlab_sign_out_direct`

Closes #33614

Signed-off-by: NRémy Coutable <remy@rymai.me>

[ci skip]

Removed all the unnecessary include of `WaitForAjax` and `ApiHelpers` in the specs.
Removed unnecessary usage of `api:true`

Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/27262

Also fixed issue with Vue resource encoding + in search term

Previously was disabled until any list was present (except for done) this now takes into account the welcome blank state

Closes #27931

Closes #23026

Closes #22710

Closes #22852

Closes #23763

Originally created to address #21197, but !6224 solved the intermittent spec
failures.

Added wait for vue helper

Vue resource is kindly encoding our URL params but Rails sadly doesn't seem to understand them.

Closes #21920

Closes #21417

Only visible when scrollable area is larger than height

Closes #21139

Closes #21071

It wasnt testing what it was saying it should be...