Skip to content

Commit 5e6725d

Browse files
author
Akos Kitta
committed
Removed the min window height and width.
Made the boards config dialog slightly smaller. Closes #216. Signed-off-by: Akos Kitta <[email protected]>
1 parent f6e623c commit 5e6725d

File tree

3 files changed

+11
-31
lines changed

3 files changed

+11
-31
lines changed

Diff for: arduino-ide-extension/src/browser/boards/boards-config-dialog.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ export class BoardsConfigDialog extends AbstractDialog<BoardsConfig.Config> {
6969
'Select both a Board and a Port if you want to upload a sketch.',
7070
'If you only select a Board you will be able just to compile, but not to upload your sketch.'
7171
]) {
72-
const p = document.createElement('p');
72+
const p = document.createElement('div');
7373
p.textContent = paragraph;
7474
text.appendChild(p);
7575
}

Diff for: arduino-ide-extension/src/browser/style/boards-config-dialog.css

+10-20
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,22 @@
11
div#select-board-dialog {
2-
margin: 5px 20px 50px 20px;
2+
margin: 5px;
33
}
44

55
div#select-board-dialog .selectBoardContainer .body {
66
display: flex;
77
overflow: hidden;
88
}
99

10-
div.dialogContent.select-board-dialog > div.head {
11-
padding-left: 21px;
10+
.select-board-dialog .head {
11+
margin: 5px;
1212
}
1313

1414
div.dialogContent.select-board-dialog > div.head .title {
1515
font-weight: 400;
1616
letter-spacing: .02em;
1717
font-size: 1.2em;
1818
color: var(--theia-arduino-branding-primary);
19-
margin: 17px 0;
20-
}
21-
22-
div#select-board-dialog .selectBoardContainer .head .text {
23-
margin-bottom: 21px;
19+
margin-bottom: 10px;
2420
}
2521

2622
div#select-board-dialog .selectBoardContainer .body .list .item.selected {
@@ -82,9 +78,9 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
8278
color: var(--theia-arduino-branding-secondary);
8379
padding: 10px 5px 10px 10px;
8480
text-transform: uppercase;
85-
/* The max, min-height comes from `.body .list` 265px + 47px top padding - 2 * 10px top padding */
86-
max-height: 292px;
87-
min-height: 292px;
81+
/* The max, min-height comes from `.body .list` 200px + 47px top padding - 2 * 10px top padding */
82+
max-height: 227px;
83+
min-height: 227px;
8884
}
8985

9086
#select-board-dialog .selectBoardContainer .body .list .item {
@@ -115,8 +111,8 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
115111
}
116112

117113
#select-board-dialog .selectBoardContainer .body .list {
118-
max-height: 265px;
119-
min-height: 265px;
114+
max-height: 200px;
115+
min-height: 200px;
120116
overflow-y: auto;
121117
}
122118

@@ -132,14 +128,8 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
132128
}
133129

134130
.p-Widget.dialogOverlay .dialogContent.select-board-dialog {
135-
width: 740px;
131+
width: 500px;
136132
}
137-
138-
139-
.dialogControl {
140-
margin: 0 20px 30px 0;
141-
}
142-
143133
.arduino-boards-toolbar-item-container {
144134
margin-left: 3px;
145135
}

Diff for: arduino-ide-extension/src/electron-main/theia/electron-main-application.ts

-10
Original file line numberDiff line numberDiff line change
@@ -82,16 +82,6 @@ export class ElectronMainApplication extends TheiaElectronMainApplication {
8282
return electronWindow;
8383
}
8484

85-
protected async getDefaultBrowserWindowOptions(): Promise<TheiaBrowserWindowOptions> {
86-
const options = await super.getDefaultBrowserWindowOptions();
87-
return {
88-
...options,
89-
// Set and use a custom minimum window size: https://github.com/arduino/arduino-pro-ide/issues/337#issuecomment-687017281
90-
minWidth: 900,
91-
minHeight: 800
92-
};
93-
}
94-
9585
protected async startBackend(): Promise<number> {
9686
// Check if we should run everything as one process.
9787
const noBackendFork = process.argv.indexOf('--no-cluster') !== -1;

0 commit comments

Comments
 (0)