Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 4e7f829

Browse files
Akos Kittakittaakos
Akos Kitta
authored andcommittedMar 31, 2021
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 26a1db3 commit 4e7f829

File tree

3 files changed

+23
-31
lines changed

3 files changed

+23
-31
lines changed
 

‎arduino-ide-extension/src/browser/boards/boards-config-dialog.ts

Lines changed: 1 addition & 1 deletion
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
}

‎arduino-ide-extension/src/browser/style/boards-config-dialog.css

Lines changed: 22 additions & 20 deletions
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 {
@@ -57,6 +53,8 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
5753
#select-board-dialog .selectBoardContainer .body .container {
5854
flex: 1;
5955
padding: 0px 10px 0px 0px;
56+
min-width: 240px;
57+
max-width: 240px;
6058
}
6159

6260
#select-board-dialog .selectBoardContainer .body .left.container .content {
@@ -82,15 +80,17 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
8280
color: var(--theia-arduino-branding-secondary);
8381
padding: 10px 5px 10px 10px;
8482
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;
83+
/* The max, min-height comes from `.body .list` 200px + 47px top padding - 2 * 10px top padding */
84+
max-height: 227px;
85+
min-height: 227px;
8886
}
8987

9088
#select-board-dialog .selectBoardContainer .body .list .item {
9189
padding: 10px 5px 10px 10px;
9290
display: flex;
9391
justify-content: end;
92+
white-space: nowrap;
93+
overflow-x: hidden;
9494
}
9595

9696
#select-board-dialog .selectBoardContainer .body .list .item .selected-icon {
@@ -114,9 +114,17 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
114114
background: var(--theia-secondaryButton-hoverBackground);
115115
}
116116

117+
#select-board-dialog .selectBoardContainer .body .list .label {
118+
max-width: 215px;
119+
width: 215px;
120+
white-space: pre;
121+
overflow: hidden;
122+
text-overflow: ellipsis;
123+
}
124+
117125
#select-board-dialog .selectBoardContainer .body .list {
118-
max-height: 265px;
119-
min-height: 265px;
126+
max-height: 200px;
127+
min-height: 200px;
120128
overflow-y: auto;
121129
}
122130

@@ -132,14 +140,8 @@ div#select-board-dialog .selectBoardContainer .body .list .item.selected i {
132140
}
133141

134142
.p-Widget.dialogOverlay .dialogContent.select-board-dialog {
135-
width: 740px;
136-
}
137-
138-
139-
.dialogControl {
140-
margin: 0 20px 30px 0;
143+
width: 500px;
141144
}
142-
143145
.arduino-boards-toolbar-item-container {
144146
margin-left: 3px;
145147
}

‎arduino-ide-extension/src/electron-main/theia/electron-main-application.ts

Lines changed: 0 additions & 10 deletions
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)