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

refactor: update the MenuBarService #292

Merged
merged 2 commits into from
Jul 29, 2021
Merged

refactor: update the MenuBarService #292

merged 2 commits into from
Jul 29, 2021

Conversation

wewoor
Copy link
Collaborator

@wewoor wewoor commented Jul 28, 2021

Description

  1. Remove the addRootMenu method, and rename initMenu to initMenus
  2. Add comments for the IMenuBarService interface

@wewoor wewoor requested a review from mortalYoung July 28, 2021 12:14
@wewoor wewoor added the refactoring Refactor label Jul 28, 2021
Copy link
Collaborator

@mortalYoung mortalYoung left a comment

Choose a reason for hiding this comment

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

+1

@mumiao mumiao merged commit da5e429 into main Jul 29, 2021
@mumiao mumiao deleted the refactor/menuBar branch July 29, 2021 11:23
ProfBramble pushed a commit that referenced this pull request Aug 4, 2021
* refactor: remove addRootMenu method, and rename initMenu to initMenus

* docs: add comments for the IMenuBarService interface
ProfBramble pushed a commit that referenced this pull request Aug 5, 2021
* refactor: remove addRootMenu method, and rename initMenu to initMenus

* docs: add comments for the IMenuBarService interface
mumiao pushed a commit that referenced this pull request Aug 5, 2021
…eys (#298)

* feat: menu bar add new file function

* feat: add encapsulated event state automata

* feat: add menu bar action to menu bar controller

* feat: define menu function and export operation id

* feat: register view shortcut

* feat: add key code string constant

* feat: register the view method in the editor service

* feat: register select all keyboard listener class

* feat: register copy line up keyboard listener class

* refactor: register undo and redo to keybinding

* perf: extract function to keybinding

* feat: add new file to keybinding

* feat: use monaco service to get menu functions

* feat: register menubar onSelect to subscribers

* refactor: update the MenuBarService (#292)

* refactor: remove addRootMenu method, and rename initMenu to initMenus

* docs: add comments for the IMenuBarService interface

* perf: move create file or folder function in service

* feat: add new file to keybinding

* feat: register menubar onSelect to subscribers

* fix: increase call rigor

* feat: add story of open any file on disk

* feat: support file highlighting

* feat: register the view method in the editor service

* perf: extract function to keybinding

* feat: add new file to keybinding

* perf: better type description

Co-authored-by: Ziv <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants