Skip to content

Conversation

kittaakos
Copy link
Contributor

Signed-off-by: Akos Kitta [email protected]

Motivation

To propose changes on #947

Change description

  • Gets rid of the _windows filed, and relies on the superclass implementation from Theia,
  • Removes redundant web-contents enablement call (done at superclass)

Other information

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@msujew, this is what I meant here. Please review. Let me know if I overlooked something, I am happy to correct it. Thank you!

@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Apr 11, 2022
@fstasi fstasi force-pushed the msujew/update-theia-1.24.0 branch 2 times, most recently from ddee44e to 8357946 Compare April 12, 2022 10:04
@msujew
Copy link
Contributor

msujew commented Apr 12, 2022

@kittaakos Thanks, the changes to the ElectronMainApplication look good to me. Due to force pushes in both of our branches, I can't easily merge it, so I just adapted your changes onto my branch. Thanks again!

@msujew msujew closed this Apr 12, 2022
@per1234 per1234 added the conclusion: duplicate Has already been submitted label Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: duplicate Has already been submitted topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants