Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Richard Knoll <[email protected]>
  • Loading branch information
thsparks and riknoll authored Mar 6, 2025
1 parent 494d9bc commit 8e956d9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion skillmap/src/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,7 @@ class AppImpl extends React.Component<AppProps, AppState> {
const prefThemeId = await authClient.getColorThemeIdAsync();
let initialTheme = this.props.highContrast
? pxt.appTarget?.appTheme?.highContrastColorTheme
: prefThemeId && this.themeManager.isKnownTheme(prefThemeId)
: (prefThemeId && this.themeManager.isKnownTheme(prefThemeId))
? prefThemeId
: pxt.appTarget?.appTheme?.defaultColorTheme;

Expand Down
2 changes: 1 addition & 1 deletion skillmap/src/lib/authClient.ts
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ export async function getBadgeStateAsync(): Promise<pxt.auth.UserBadgeState | un

export async function getColorThemeIdAsync(): Promise<string | undefined> {
const prefs = await userPreferencesAsync();
if(prefs) {
if (prefs) {
return prefs?.colorThemeIds?.[pxt.appTarget.id];
}
}
Expand Down

0 comments on commit 8e956d9

Please sign in to comment.