Skip to content

Commit 0bf73a1

Browse files
committed
fixes after code review
1 parent cf3445a commit 0bf73a1

File tree

4 files changed

+13
-4
lines changed

4 files changed

+13
-4
lines changed

arduino-ide-extension/src/browser/arduino-frontend-contribution.tsx

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -418,6 +418,15 @@ export class ArduinoFrontendContribution implements FrontendApplicationContribut
418418
},
419419
description: 'Background color of the toolbar items when hovering over them. Such as Upload, Verify, etc.'
420420
},
421+
{
422+
id: 'arduino.toolbar.toggleBackground',
423+
defaults: {
424+
dark: 'editor.selectionBackground',
425+
light: 'editor.selectionBackground',
426+
hc: 'activityBar.inactiveForeground'
427+
},
428+
description: 'Toggle color of the toolbar items when they are currently toggled (the command is in progress)'
429+
},
421430
{
422431
id: 'arduino.output.foreground',
423432
defaults: {

arduino-ide-extension/src/browser/contributions/upload-sketch.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ export class UploadSketch extends SketchContribution {
2222
@inject(BoardsServiceProvider)
2323
protected readonly boardsServiceClientImpl: BoardsServiceProvider;
2424

25-
uploadInProgress = false;
25+
protected uploadInProgress = false;
2626

2727
registerCommands(registry: CommandRegistry): void {
2828
registry.registerCommand(UploadSketch.Commands.UPLOAD_SKETCH, {

arduino-ide-extension/src/browser/contributions/verify-sketch.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ export class VerifySketch extends SketchContribution {
1818
@inject(BoardsServiceProvider)
1919
protected readonly boardsServiceClientImpl: BoardsServiceProvider;
2020

21-
verifyInProgress = false;
21+
protected verifyInProgress = false;
2222

2323
registerCommands(registry: CommandRegistry): void {
2424
registry.registerCommand(VerifySketch.Commands.VERIFY_SKETCH, {

arduino-ide-extension/src/browser/style/main.css

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
}
1717

1818
.p-TabBar-toolbar .item.arduino-tool-item:hover > div {
19-
background: white;
19+
background: var(--theia-arduino-toolbar-hoverBackground);
2020
}
2121

2222
.arduino-verify-sketch--toolbar,
@@ -30,7 +30,7 @@
3030
}
3131

3232
.item.arduino-tool-item.toggled .arduino-verify-sketch--toolbar {
33-
background-color: rgb(255,204,0) !important;
33+
background-color: var(--theia-arduino-toolbar-toggleBackground) !important;
3434
}
3535

3636
.arduino-tool-icon {

0 commit comments

Comments
 (0)