BUG: Fix DataFrame logical ops Series inconsistency
closes #5284 
closes #5035
- [x] tests added / passed
- [x] passes `black pandas`
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [x] whatsnew entry

Will need to look and see if this closes anything else.

This will have a merge conflict with #28638, that is a slightly higher priority.

Not sure if this constitutes an API change as the behavior for these ops is all over the place.

