Skip to content

Commit 70125a2

Browse files
authored
Update get started theme picker images and styles (microsoft#166651)
Update theme picker images and styles
1 parent 53ccc82 commit 70125a2

File tree

6 files changed

+38
-21
lines changed

6 files changed

+38
-21
lines changed

src/vs/workbench/contrib/welcomeGettingStarted/browser/gettingStartedDetailsRenderer.ts

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,15 @@ export class GettingStartedDetailsRenderer {
6060
padding: 0;
6161
height: inherit;
6262
}
63+
.theme-picker-row {
64+
display: flex;
65+
gap: 32px;
66+
justify-content: center;
67+
}
6368
checklist {
6469
display: flex;
65-
flex-wrap: wrap;
66-
justify-content: space-around;
70+
gap: 32px;
71+
flex-direction: column;
6772
}
6873
checkbox {
6974
display: flex;
@@ -73,14 +78,19 @@ export class GettingStartedDetailsRenderer {
7378
cursor: pointer;
7479
}
7580
checkbox > img {
76-
margin-bottom: 4px;
81+
margin-bottom: 8px !important;
7782
}
7883
checkbox.checked > img {
7984
box-sizing: border-box;
8085
}
8186
checkbox.checked > img {
8287
outline: 2px solid var(--vscode-focusBorder);
83-
outline-offset: 2px;
88+
outline-offset: 4px;
89+
border-radius: 4px;
90+
}
91+
.theme-picker-link {
92+
margin-top: 16px;
93+
color: var(--vscode-textLink-foreground);
8494
}
8595
blockquote > p:first-child {
8696
margin-top: 0;
@@ -92,6 +102,9 @@ export class GettingStartedDetailsRenderer {
92102
vertically-centered {
93103
padding-top: 5px;
94104
padding-bottom: 5px;
105+
display: flex;
106+
justify-content: center;
107+
flex-direction: column;
95108
}
96109
html {
97110
height: 100%;
8.17 KB
Loading
8.51 KB
Loading
8.2 KB
Loading
7.98 KB
Loading

src/vs/workbench/contrib/welcomeGettingStarted/common/media/theme_picker.ts

Lines changed: 21 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -8,24 +8,28 @@ import { localize } from 'vs/nls';
88

99
export default () => `
1010
<checklist>
11-
<checkbox when-checked="setTheme:Default Light+" checked-on="config.workbench.colorTheme == 'Default Light+'">
12-
<img width="150" src="./light.png"/>
13-
${escape(localize('light', "Light"))}
14-
</checkbox>
15-
<checkbox when-checked="setTheme:Default Dark+" checked-on="config.workbench.colorTheme == 'Default Dark+'">
16-
<img width="150" src="./dark.png"/>
17-
${escape(localize('dark', "Dark"))}
18-
</checkbox>
19-
<checkbox when-checked="setTheme:Default High Contrast" checked-on="config.workbench.colorTheme == 'Default High Contrast'">
20-
<img width="150" src="./dark-hc.png"/>
21-
${escape(localize('HighContrast', "Dark High Contrast"))}
22-
</checkbox>
23-
<checkbox when-checked="setTheme:Default High Contrast Light" checked-on="config.workbench.colorTheme == 'Default High Contrast Light'">
24-
<img width="150" src="./light-hc.png"/>
25-
${escape(localize('HighContrastLight', "Light High Contrast"))}
26-
</checkbox>
11+
<div class="theme-picker-row">
12+
<checkbox when-checked="setTheme:Default Dark+" checked-on="config.workbench.colorTheme == 'Default Dark+'">
13+
<img width="200" src="./dark.png"/>
14+
${escape(localize('dark', "Dark"))}
15+
</checkbox>
16+
<checkbox when-checked="setTheme:Default Light+" checked-on="config.workbench.colorTheme == 'Default Light+'">
17+
<img width="200" src="./light.png"/>
18+
${escape(localize('light', "Light"))}
19+
</checkbox>
20+
</div>
21+
<div class="theme-picker-row">
22+
<checkbox when-checked="setTheme:Default High Contrast" checked-on="config.workbench.colorTheme == 'Default High Contrast'">
23+
<img width="200" src="./dark-hc.png"/>
24+
${escape(localize('HighContrast', "Dark High Contrast"))}
25+
</checkbox>
26+
<checkbox when-checked="setTheme:Default High Contrast Light" checked-on="config.workbench.colorTheme == 'Default High Contrast Light'">
27+
<img width="200" src="./light-hc.png"/>
28+
${escape(localize('HighContrastLight', "Light High Contrast"))}
29+
</checkbox>
30+
</div>
2731
</checklist>
28-
<checkbox when-checked="command:workbench.action.selectTheme" checked-on="false">
32+
<checkbox class="theme-picker-link" when-checked="command:workbench.action.selectTheme" checked-on="false">
2933
${escape(localize('seeMore', "See More Themes..."))}
3034
</checkbox>
3135
`;

0 commit comments

Comments
 (0)