Reorganize context menus

[katelynienaber]

Signed-off-by: Katelyn Nienaber katelyn.nienaber@broadcom.com

Proposed changes

Rearranged context menus & toolbar commands according to issues #347 and #765

Release Notes

Milestone: Backlog

Changelog: Rearranged context menus & toolbar commands

Types of changes

What types of changes does your code introduce to Zowe Explorer?
Put an x in the boxes that apply

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Updates to Documentation or Tests (if none of the other choices apply)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This checklist will be used as reference for both the contributor and the reviewer

• I have read the CONTRIBUTOR GUIDANCE wiki
• PR title follows Conventional Commits Guidelines
• PR Description is included
• npm run vscode:prepublish has been executed
• All checks have passed (DCO, Jenkins and Code Coverage)
• I have added unit test and it is passing
• I have added integration test and it is passing
• There is coverage for the code that I have added
• I have tested it manually and there are no regressions found
• I have added necessary documentation (if appropriate)
• Any PR dependencies have been merged and published (if appropriate)

[codecov]

Codecov Report

Merging #834 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #834   +/-   ##
=======================================
  Coverage   90.09%   90.09%           
=======================================
  Files          47       47           
  Lines        4331     4331           
  Branches      765      765           
=======================================
  Hits         3902     3902           
  Misses        425      425           
  Partials        4        4           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03ca792...50d4dbe. Read the comment docs.

[Pranay154]

@katelynienaber Thanks for doing this! Looks great 👍 Just the one comment I've left.

[lauren-li]

@katelynienaber Thank you for your work on this!

For Data Sets, could we keep the "Delete" group of commands at the bottom, as that's how VS Code does their context menus? "Migrate" and "Recall" are currently numbered to show up below "Delete" and its related commands.

[Pranay154]

Im happy with the changes, thanks for doing this 😄

[katelynienaber]

@katelynienaber Thank you for your work on this!

For Data Sets, could we keep the "Delete" group of commands at the bottom, as that's how VS Code does their context menus? "Migrate" and "Recall" are currently numbered to show up below "Delete" and its related commands.

This should be done now :) Was just waiting for Jenkins to cooperate...

[lauren-li]

Thanks @katelynienaber for considering my suggestion! This looks good to me now. All unit and system tests are passing, and functional testing seems to work well without any regressions. I appreciate that the context menu organization feels closer to VS Code's file explorer context menus with this PR.

[VitGottwald]

LGTM. The only comment I have is that we should let people know we changed stuff. Pictures would be best. I wonder if we have a way to deliver such an update.