Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

explorer: update toolbar items for new file and new folder #11429

Merged
merged 1 commit into from
Jul 16, 2022

Conversation

vince-fugnitto
Copy link
Member

What it does

Closes: #11427.
Closes: #11425.

The pull-request adds dedicated items for both New File and New Folder in the Explorer.

Screen Shot 2022-07-14 at 3 53 03 PM

How to test

  1. start the application with a workspace present.
  2. confirm that F1 > New File triggers the new file with a dialog.
  3. confirm that F1 > New Untitled File opens an untitled file.
  4. confirm that there are no duplicate entries for New File and New Folder in the command palette.
  5. confirm that New File from the top-level File menu opens an untitled file.
  6. confirm that the New File toolbar item exists in the Explorer - triggers a new file with a dialog.
  7. confirm that the New Folder toolbar item exists in the Explorer - triggers a new folder with a dialog.
  8. close the workspace - confirm that both New File and New Folder do not exist in the Explorer.

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com
Co-authored-by: yiliang114 1204183885@qq.com

@vince-fugnitto vince-fugnitto added the navigator issues related to the navigator/explorer label Jul 14, 2022
@vince-fugnitto vince-fugnitto self-assigned this Jul 14, 2022
Copy link
Contributor

@colin-grant-work colin-grant-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The behavior is consistent with the testing steps and VSCode, and the command palette is clean with no duplicate items with the same label. 👍

The commit includes the following updates:
- adds a `New File` toolbar item to replace the entry under the more context-menu.
- adds a `New Folder` toolbar item to replace the entry under the more context-menu.
- renamed `CommonCommands.NEW_FILE` to `CommonCommands.NEW_UNTITLED_FILE`.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
Co-authored-by: yiliang114 <1204183885@qq.com>
Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me as well 👍

@vince-fugnitto vince-fugnitto merged commit f3c48a9 into master Jul 16, 2022
@vince-fugnitto vince-fugnitto deleted the vf/explorer-toolbar branch July 16, 2022 01:27
@github-actions github-actions bot added this to the 1.28.0 milestone Jul 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
navigator issues related to the navigator/explorer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NEW_FILE and NEW_FOLDER in front of navigator toolbar should show with icons like vscode
3 participants