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

electron: remove disallowReloadKeybinding #11099

Closed
vince-fugnitto opened this issue May 3, 2022 · 0 comments · Fixed by #11174
Closed

electron: remove disallowReloadKeybinding #11099

vince-fugnitto opened this issue May 3, 2022 · 0 comments · Fixed by #11174
Labels
beginners issues that are perfect for beginners good first issue good first issues for new contributors

Comments

@vince-fugnitto
Copy link
Member

vince-fugnitto commented May 3, 2022

Feature Description:

The application configuration option disallowReloadKeybinding is no longer used, and can be deprecated/removed:

/**
* If set to `true`, reloading the current browser window won't be possible with the `Ctrl/Cmd + R` keybinding.
*
* Has no effect if not in an electron environment.
*
* Defaults to `false`.
*/
readonly disallowReloadKeybinding?: boolean;

It is currently only used in a test, where i can be safely removed:

it('should use defaults when calling `set`', function (): void {
FrontendApplicationConfigProvider.set({
applicationName: DEFAULT.applicationName + ' Something Else',
electron: {
disallowReloadKeybinding: !DEFAULT.electron.disallowReloadKeybinding
}
});
const config = FrontendApplicationConfigProvider.get();
// custom values
expect(config.applicationName).not.equal(DEFAULT.applicationName);
expect(config.electron.disallowReloadKeybinding).not.equal(DEFAULT.electron.disallowReloadKeybinding);
// defaults
expect(config.defaultIconTheme).equal(DEFAULT.defaultIconTheme);
expect(config.defaultTheme).equal(DEFAULT.defaultTheme);
expect(config.electron.windowOptions).deep.equal(DEFAULT.electron.windowOptions);
});

Related Info:

@vince-fugnitto vince-fugnitto added beginners issues that are perfect for beginners good first issue good first issues for new contributors labels May 3, 2022
kittaakos pushed a commit to arduino/arduino-ide that referenced this issue May 18, 2022
Ref: eclipse-theia/theia#11099
Signed-off-by: Akos Kitta <a.kitta@arduino.cc>
ShehanAT added a commit to ShehanAT/theia that referenced this issue May 18, 2022
…ation-package/src/application-props.ts; fixes Issue eclipse-theia#11099; Signed-off by Shehan Atukorala<shehanatuk@gmail.com>
ShehanAT added a commit to ShehanAT/theia that referenced this issue May 18, 2022
…ation-package/src/application-props.ts; fixes Issue eclipse-theia#11099; Signed-off by Shehan Atukorala<shehanatuk@gmail.com>
ShehanAT added a commit to ShehanAT/theia that referenced this issue May 19, 2022
…ation-package/src/application-props.ts; fixes Issue eclipse-theia#11099; Signed-off by Shehan Atukorala<shehanatuk@gmail.com>
ShehanAT added a commit to ShehanAT/theia that referenced this issue May 19, 2022
…ation-package/src/application-props.ts; fixes Issue eclipse-theia#11099; Signed-off by Shehan Atukorala<shehanatuk@gmail.com>
ShehanAT added a commit to ShehanAT/theia that referenced this issue May 19, 2022
…cation-package/src/application-props.ts; fixes Issue eclipse-theia#11099; Signed-off by Sean <shehanatuk@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beginners issues that are perfect for beginners good first issue good first issues for new contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant