Move NavigationShortcutsBlock into the Shortcut module.

Moved the block in shortcut module along with respective test case.

Please review. moving NR.

Small comment.

Addressed the feedback, Please review moving NR

1 comment addressed.

Shortcut module still works after this change, looks good to me.

I would like to point out that the block id is still prefixed with navigation_. Not sure if this is what we want. https://git.drupalcode.org/project/drupal/-/merge_requests/9066/diffs#f1...

If we change it, we may need to write a database update.

I'm marking this as needs review in case my comments should be considered as part of this MR. Feel free to change it back to Reviewed & tested by the community if my comments don't need consideration, sorry for the additional work.

Don't think the ID needs to change but block does actually need an update. Currently has a check for shortcut but being in the shortcut module this isn't needed but should check navigation.

As per feedback, added navigation module check along with comment also updated, not sure apart from it, nothing seems to be left.

Please review, moving NR.

Not sure if we will have to move the service too but currently as scoped believe this is good .

ShortcutLazyBuilder should be moved in this issue too, it only exists to support the block.

I also think we might want to wait to commit this one until the hard blockers for navigation to become stable are committed or at least close to RTBC - otherwise this block will be showing up on the UI on modules without navigation installed while it's still experimental.