Skip to content

Commit 74440a1

Browse files
authored
updates styles (#10652)
* updates styles * fix summary section overflow * fix(ui): remove duplicated max-width
1 parent caa1b38 commit 74440a1

File tree

9 files changed

+274
-171
lines changed

9 files changed

+274
-171
lines changed

apps/desktop/src/components/BranchHeaderIcon.svelte

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,13 @@
66
iconName: keyof typeof iconsJson;
77
color: string;
88
small?: boolean;
9+
large?: boolean;
910
}
1011
11-
const { iconName, color, small }: Props = $props();
12+
const { iconName, color, small, large }: Props = $props();
1213
</script>
1314

14-
<div class="stack__status--icon" style:--bg-color={color} class:small>
15+
<div class="stack__status--icon" style:--bg-color={color} class:small class:large>
1516
<Icon name={iconName} />
1617
</div>
1718

@@ -21,18 +22,25 @@
2122
flex-shrink: 0;
2223
align-items: center;
2324
justify-content: center;
24-
border-radius: var(--radius-m);
2525
background-color: var(--bg-color);
2626
color: #fff;
2727
28-
&:not(.small) {
28+
&:not(.small):not(.large) {
2929
width: 20px;
3030
height: 20px;
31+
border-radius: var(--radius-m);
3132
}
3233
3334
&.small {
3435
width: 18px;
3536
height: 18px;
37+
border-radius: var(--radius-m);
38+
}
39+
40+
&.large {
41+
width: var(--size-button);
42+
height: var(--size-button);
43+
border-radius: var(--radius-ml);
3644
}
3745
}
3846
</style>

apps/desktop/src/components/codegen/CodegenChatLayout.svelte

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -37,17 +37,19 @@
3737
<div class="chat" use:focusable={{ vertical: true }}>
3838
<div class="chat-header" use:focusable>
3939
<div class="flex gap-10 justify-between">
40+
{@render branchIcon()}
41+
4042
<div class="chat-header__title">
41-
{@render branchIcon()}
42-
<p class="text-15 text-bold truncate">{branchName}</p>
43-
</div>
44-
<div class="flex gap-4 items-center">
45-
{@render contextActions()}
43+
<h3 class="text-15 text-bold truncate">{branchName}</h3>
44+
45+
<div class="chat-header__title-actions">
46+
{@render workspaceActions()}
47+
</div>
4648
</div>
4749
</div>
4850

49-
<div class="chat-header__actions">
50-
{@render workspaceActions()}
51+
<div class="flex gap-8">
52+
{@render contextActions()}
5153
</div>
5254
</div>
5355

@@ -87,23 +89,21 @@
8789
8890
.chat-header {
8991
display: flex;
90-
flex-direction: column;
92+
justify-content: space-between;
9193
width: 100%;
9294
padding: 16px;
9395
gap: 8px;
94-
border-bottom: 1px solid var(--clr-border-3);
96+
border-bottom: 1px solid var(--clr-border-2);
9597
}
9698
9799
.chat-header__title {
98100
display: flex;
99-
flex: 1;
100-
align-items: center;
101-
margin-top: -2px;
101+
flex-direction: column;
102102
overflow: hidden;
103-
gap: 10px;
103+
gap: 8px;
104104
}
105105
106-
.chat-header__actions {
106+
.chat-header__title-actions {
107107
display: flex;
108108
align-items: center;
109109
gap: 4px;

apps/desktop/src/components/codegen/CodegenClaudeMessage.svelte

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<script lang="ts">
22
import CodegenAssistantMessage from '$components/codegen/CodegenAssistantMessage.svelte';
3+
import CodegenServiceMessage from '$components/codegen/CodegenServiceMessage.svelte';
34
import CodegenToolCall from '$components/codegen/CodegenToolCall.svelte';
45
import CodegenToolCalls from '$components/codegen/CodegenToolCalls.svelte';
56
import CodegenUserMessage from '$components/codegen/CodegenUserMessage.svelte';
@@ -21,11 +22,11 @@
2122
<CodegenUserMessage content={message.message} avatarUrl={userAvatarUrl} />
2223
{:else if message.type === 'claude'}
2324
{#if 'subtype' in message && message.subtype === 'compaction'}
24-
<CodegenAssistantMessage content="The conversation has been compacted.">
25+
<CodegenServiceMessage style="neutral" face="compacted" reverseElementsOrder>
2526
{#snippet extraContent()}
2627
{@render compactionSummary(message.message)}
2728
{/snippet}
28-
</CodegenAssistantMessage>
29+
</CodegenServiceMessage>
2930
{:else}
3031
<CodegenAssistantMessage content={message.message}>
3132
{#snippet extraContent()}
@@ -51,7 +52,6 @@
5152
{#snippet compactionSummary(summary: string)}
5253
<div class="compaction-summary__wrapper">
5354
<div class="compaction-summary" class:expanded>
54-
<!-- Header for multiple tool calls -->
5555
<button
5656
type="button"
5757
class="compaction-summary__header"
@@ -60,10 +60,10 @@
6060
<div class="compaction-summary__arrow" class:expanded>
6161
<Icon name="chevron-right" />
6262
</div>
63-
<span class="text-13 text-semibold">Compaction summary</span>
63+
<p class="text-13 text-italic clr-text-2">Conversation compacted to preserve context</p>
6464
</button>
6565
{#if expanded}
66-
<div class="compaction-summary__content">
66+
<div class="text-13 compaction-summary__content">
6767
<Markdown content={summary} />
6868
</div>
6969
{/if}
@@ -73,14 +73,11 @@
7373

7474
<style lang="postcss">
7575
.compaction-summary__wrapper {
76-
container-name: assistant-message;
77-
container-type: inline-size;
76+
max-width: var(--message-max-width);
7877
}
7978
8079
.compaction-summary {
8180
width: fit-content;
82-
max-width: var(--message-max-width);
83-
max-width: 100%;
8481
overflow: hidden;
8582
border: 1px solid var(--clr-border-2);
8683
border-radius: var(--radius-ml);
@@ -94,14 +91,14 @@
9491
display: flex;
9592
align-items: center;
9693
width: 100%;
97-
padding: 8px 12px 8px 8px;
94+
padding: 10px 12px 10px 8px;
9895
gap: 8px;
99-
border: none;
96+
background-color: var(--clr-bg-2);
10097
cursor: pointer;
10198
transition: background-color var(--transition-fast);
10299
103100
&:hover {
104-
background-color: var(--clr-bg-1-muted);
101+
background-color: var(--clr-bg-2-muted);
105102
106103
.compaction-summary__arrow {
107104
color: var(--clr-text-2);
@@ -112,9 +109,7 @@
112109
.compaction-summary__arrow {
113110
display: flex;
114111
color: var(--clr-text-3);
115-
transition:
116-
,
117-
transform var(--transition-medium);
112+
transition: transform var(--transition-medium);
118113
119114
&.expanded {
120115
transform: rotate(90deg);

0 commit comments

Comments
 (0)