Skip to content

Commit c6eeb95

Browse files
committed
- UI Glam Session -> Makeover for TaskHeader, ChatView, HistoryPreview, WelcomeView
- In particular, display a "no tasks in workspace" message when no tasks are found. - Clean up Inferface Settings (not needed now) on Settings View - Copy updates throughout these areas.
1 parent 30c44ff commit c6eeb95

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

70 files changed

+273
-591
lines changed

assets/images/roo-logo.svg

Lines changed: 3 additions & 0 deletions
Loading

src/core/webview/ClineProvider.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -655,7 +655,7 @@ export class ClineProvider extends EventEmitter<ClineProviderEvents> implements
655655
<link rel="stylesheet" type="text/css" href="${stylesUri}">
656656
<link href="${codiconsUri}" rel="stylesheet" />
657657
<script nonce="${nonce}">
658-
window.IMAGES_BASE_URI = "${imagesUri}"
658+
window.IMAGES_BASE_URI = "${imagesUri}"
659659
</script>
660660
<title>Roo Code</title>
661661
</head>
@@ -1216,7 +1216,6 @@ export class ClineProvider extends EventEmitter<ClineProviderEvents> implements
12161216
telemetrySetting,
12171217
showRooIgnoredFiles,
12181218
language,
1219-
showGreeting,
12201219
maxReadFileLine,
12211220
} = await this.getState()
12221221

@@ -1297,7 +1296,6 @@ export class ClineProvider extends EventEmitter<ClineProviderEvents> implements
12971296
renderContext: this.renderContext,
12981297
maxReadFileLine: maxReadFileLine ?? 500,
12991298
settingsImportedAt: this.settingsImportedAt,
1300-
showGreeting: showGreeting ?? true, // Ensure showGreeting is included in the returned state
13011299
}
13021300
}
13031301

@@ -1385,7 +1383,6 @@ export class ClineProvider extends EventEmitter<ClineProviderEvents> implements
13851383
telemetrySetting: stateValues.telemetrySetting || "unset",
13861384
showRooIgnoredFiles: stateValues.showRooIgnoredFiles ?? true,
13871385
maxReadFileLine: stateValues.maxReadFileLine ?? 500,
1388-
showGreeting: stateValues.showGreeting ?? true, // Ensure showGreeting is returned by getState
13891386
}
13901387
}
13911388

src/core/webview/webviewMessageHandler.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -645,11 +645,6 @@ export const webviewMessageHandler = async (provider: ClineProvider, message: We
645645
await updateGlobalState("diffEnabled", diffEnabled)
646646
await provider.postStateToWebview()
647647
break
648-
case "showGreeting":
649-
const showGreeting = message.bool ?? true
650-
await updateGlobalState("showGreeting", showGreeting)
651-
await provider.postStateToWebview()
652-
break
653648
case "enableCheckpoints":
654649
const enableCheckpoints = message.bool ?? true
655650
await updateGlobalState("enableCheckpoints", enableCheckpoints)

src/exports/roo-code.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ type GlobalSettings = {
262262
remoteBrowserHost?: string | undefined
263263
cachedChromeHostUrl?: string | undefined
264264
enableCheckpoints?: boolean | undefined
265-
showGreeting?: boolean | undefined
265+
checkpointStorage?: ("task" | "workspace") | undefined
266266
ttsEnabled?: boolean | undefined
267267
ttsSpeed?: number | undefined
268268
soundEnabled?: boolean | undefined

src/exports/types.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@ type GlobalSettings = {
265265
remoteBrowserHost?: string | undefined
266266
cachedChromeHostUrl?: string | undefined
267267
enableCheckpoints?: boolean | undefined
268-
showGreeting?: boolean | undefined
268+
checkpointStorage?: ("task" | "workspace") | undefined
269269
ttsEnabled?: boolean | undefined
270270
ttsSpeed?: number | undefined
271271
soundEnabled?: boolean | undefined

src/schemas/index.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -529,8 +529,6 @@ export const globalSettingsSchema = z.object({
529529

530530
enableCheckpoints: z.boolean().optional(),
531531

532-
showGreeting: z.boolean().optional(),
533-
534532
ttsEnabled: z.boolean().optional(),
535533
ttsSpeed: z.number().optional(),
536534
soundEnabled: z.boolean().optional(),
@@ -606,8 +604,6 @@ const globalSettingsRecord: GlobalSettingsRecord = {
606604

607605
enableCheckpoints: undefined,
608606

609-
showGreeting: undefined,
610-
611607
ttsEnabled: undefined,
612608
ttsSpeed: undefined,
613609
soundEnabled: undefined,

src/shared/ExtensionMessage.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,6 @@ export type ExtensionState = Pick<
142142
| "remoteBrowserEnabled"
143143
| "remoteBrowserHost"
144144
// | "enableCheckpoints" // Optional in GlobalSettings, required here.
145-
| "showGreeting"
146145
| "ttsEnabled"
147146
| "ttsSpeed"
148147
| "soundEnabled"

src/shared/WebviewMessage.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,6 @@ export interface WebviewMessage {
125125
| "maxReadFileLine"
126126
| "searchFiles"
127127
| "toggleApiConfigPin"
128-
| "showGreeting"
129128
text?: string
130129
disabled?: boolean
131130
askResponse?: ClineAskResponse

webview-ui/src/components/chat/ChatRow.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -854,11 +854,10 @@ export const ChatRowContent = ({
854854
case "user_feedback":
855855
return (
856856
<div
857+
className="outline rounded p-4"
857858
style={{
858-
backgroundColor: "var(--vscode-badge-background)",
859859
color: "var(--vscode-badge-foreground)",
860-
borderRadius: "3px",
861-
padding: "9px",
860+
padding: "4px",
862861
overflow: "hidden",
863862
whiteSpace: "pre-wrap",
864863
wordBreak: "break-word",

webview-ui/src/components/chat/ChatView.tsx

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import { getApiMetrics } from "../../../../src/shared/getApiMetrics"
1919
import { useExtensionState } from "../../context/ExtensionStateContext"
2020
import { vscode } from "../../utils/vscode"
2121
import HistoryPreview from "../history/HistoryPreview"
22+
import RooHero from "../welcome/RooHero"
2223
import { normalizeApiConfiguration } from "../settings/ApiOptions"
2324
import Announcement from "./Announcement"
2425
import BrowserSessionRow from "./BrowserSessionRow"
@@ -76,7 +77,6 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
7677
alwaysAllowSubtasks,
7778
customModes,
7879
telemetrySetting,
79-
showGreeting,
8080
} = useExtensionState()
8181

8282
//const task = messages.length > 0 ? (messages[0].say === "task" ? messages[0] : undefined) : undefined) : undefined
@@ -1224,12 +1224,8 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
12241224
}}>
12251225
{telemetrySetting === "unset" && <TelemetryBanner />}
12261226
{showAnnouncement && <Announcement version={version} hideAnnouncement={hideAnnouncement} />}
1227-
{showGreeting === true && (
1228-
<div style={{ padding: "0 20px", flexShrink: 0 }}>
1229-
<h2>{t("chat:greeting")}</h2>
1230-
<p>{t("chat:aboutMe")}</p>
1231-
</div>
1232-
)}
1227+
1228+
<RooHero />
12331229
{taskHistory.length > 0 && <HistoryPreview showHistoryView={showHistoryView} />}
12341230
</div>
12351231
)}

0 commit comments

Comments
 (0)