I'm confused about adding raw support in this PR, is there a reason to do that here instead of as a separate PR?

I'm confused about adding raw support in this PR, is there a reason to do that here instead of as a separate PR?

Good point. I initially thought that adding raw support for shift would be necessary to fix since gshift supports raw (so gshift and shift stay consistent). Escaping gforce in this case and deactivating gshift support for raw might be the better route.

I'm confused about adding raw support in this PR, is there a reason to do that here instead of as a separate PR?

Good point. I initially thought that adding raw support for shift would be necessary to fix since gshift supports raw (so gshift and shift stay consistent). Escaping gforce in this case and deactivating gshift support for raw might be the better route.

Oh, I see. I edited the PR description with my understanding. It's fine to keep the change in this PR IIUC.

Fixed the trivial bits. Pending things are:

• Can we simplify is_constantish() as suggested?
• Updating the NEWS to reflect the current state of the PR

Done. I hope the new NEWS item reflects the changes enough.

Great work!

Remarking just that if this PR closes #5547, taking into account #5680 (comment)

Ok this is the same as #5547

then it also closes #5680 (which is currently open)