From bfcb518174a78912e49e9a9fb1fc0b2eed5d1a43 Mon Sep 17 00:00:00 2001 From: Akos Kitta Date: Wed, 23 Nov 2022 14:16:48 +0100 Subject: [PATCH] fix: revive `--theia-icon-loading` in IDE2 - it has been removed from Theia - aligned the DOM structure to previous the version when using widget Signed-off-by: Akos Kitta --- .../firmware-uploader-dialog.tsx | 25 ++++++++----------- .../src/browser/icons/loading-dark.svg | 6 +++++ .../src/browser/icons/loading-light.svg | 6 +++++ .../src/browser/style/index.css | 10 ++++++++ 4 files changed, 33 insertions(+), 14 deletions(-) create mode 100644 arduino-ide-extension/src/browser/icons/loading-dark.svg create mode 100644 arduino-ide-extension/src/browser/icons/loading-light.svg diff --git a/arduino-ide-extension/src/browser/dialogs/firmware-uploader/firmware-uploader-dialog.tsx b/arduino-ide-extension/src/browser/dialogs/firmware-uploader/firmware-uploader-dialog.tsx index c8ae0b69d..20d97d4b2 100644 --- a/arduino-ide-extension/src/browser/dialogs/firmware-uploader/firmware-uploader-dialog.tsx +++ b/arduino-ide-extension/src/browser/dialogs/firmware-uploader/firmware-uploader-dialog.tsx @@ -67,15 +67,17 @@ export class UploadFirmwareDialog extends ReactDialog { protected override render(): React.ReactNode { return ( -
- - +
+
+ + +
); } @@ -86,11 +88,6 @@ export class UploadFirmwareDialog extends ReactDialog { this.update(); } - protected override onUpdateRequest(msg: Message): void { - super.onUpdateRequest(msg); - this.update(); - } - // eslint-disable-next-line unused-imports/no-unused-vars, @typescript-eslint/no-unused-vars protected override handleEnter(event: KeyboardEvent): boolean | void { return false; diff --git a/arduino-ide-extension/src/browser/icons/loading-dark.svg b/arduino-ide-extension/src/browser/icons/loading-dark.svg new file mode 100644 index 000000000..d886fd06f --- /dev/null +++ b/arduino-ide-extension/src/browser/icons/loading-dark.svg @@ -0,0 +1,6 @@ + + + + + + diff --git a/arduino-ide-extension/src/browser/icons/loading-light.svg b/arduino-ide-extension/src/browser/icons/loading-light.svg new file mode 100644 index 000000000..d46f25880 --- /dev/null +++ b/arduino-ide-extension/src/browser/icons/loading-light.svg @@ -0,0 +1,6 @@ + + + + + + diff --git a/arduino-ide-extension/src/browser/style/index.css b/arduino-ide-extension/src/browser/style/index.css index e46cdcdd7..638403697 100644 --- a/arduino-ide-extension/src/browser/style/index.css +++ b/arduino-ide-extension/src/browser/style/index.css @@ -20,6 +20,16 @@ @import './progress-bar.css'; @import './settings-step-input.css'; +/* Revive of the `--theia-icon-loading`. The variable has been removed from Theia while IDE2 still uses is. */ +/* The SVG icons are still part of Theia (1.31.1) */ +/* https://github.com/arduino/arduino-ide/pull/1662#issuecomment-1324997134 */ +body { + --theia-icon-loading: url(../icons/loading-light.svg); +} +body.theia-dark { + --theia-icon-loading: url(../icons/loading-dark.svg); +} + .theia-input.warning:focus { outline-width: 1px; outline-style: solid;