Skip to content

Commit 89fbf78

Browse files
committed
feat(web): align mobile files content with pc sidebar
1 parent 5aff084 commit 89fbf78

2 files changed

Lines changed: 67 additions & 30 deletions

File tree

packages/web/src/styles/components.css

Lines changed: 22 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -10105,11 +10105,9 @@ textarea.input {
1010510105
.mobile-sheet--files .workspace-git-view {
1010610106
overflow: hidden;
1010710107
border: 1px solid color-mix(in srgb, var(--border) 80%, transparent);
10108-
border-radius: var(--radius-xl) var(--radius-xl) 0 0;
10109-
background: color-mix(in srgb, var(--bg-surface) 90%, var(--bg-page));
10110-
box-shadow:
10111-
inset 0 1px 0 color-mix(in srgb, var(--fg-white) 5%, transparent),
10112-
0 14px 34px color-mix(in srgb, var(--bg-page) 18%, transparent);
10108+
border-radius: var(--radius-md);
10109+
background: var(--bg-panel);
10110+
box-shadow: none;
1011310111
}
1011410112

1011510113
.mobile-sheet--files .workspace-git-view {
@@ -11583,7 +11581,7 @@ textarea.input {
1158311581
}
1158411582

1158511583
.file-tree-shell--mobile .file-tree-search {
11586-
margin: 0 0 10px;
11584+
margin: 0;
1158711585
border-right: none;
1158811586
border-left: none;
1158911587
border-radius: 0;
@@ -11604,6 +11602,8 @@ textarea.input {
1160411602
}
1160511603

1160611604
.file-tree-shell--mobile .tree-item.selected {
11605+
padding-left: 14px;
11606+
border-left: 2px solid color-mix(in srgb, var(--accent-blue) 88%, white 12%);
1160711607
background: color-mix(in srgb, var(--accent-blue) 12%, transparent);
1160811608
}
1160911609

@@ -12114,17 +12114,11 @@ textarea.input {
1211412114
align-items: center;
1211512115
flex-shrink: 0;
1211612116
gap: var(--sp-3);
12117-
padding: var(--sp-3) var(--sp-3) var(--sp-2);
12118-
border: 1px solid color-mix(in srgb, var(--border) 78%, transparent);
12119-
border-radius: 18px;
12120-
background: linear-gradient(
12121-
180deg,
12122-
color-mix(in srgb, var(--bg-surface) 92%, transparent) 0%,
12123-
color-mix(in srgb, var(--bg-page) 92%, var(--bg-surface) 8%) 100%
12124-
);
12125-
box-shadow:
12126-
inset 0 1px 0 color-mix(in srgb, var(--fg-white) 5%, transparent),
12127-
0 10px 24px color-mix(in srgb, var(--bg-page) 16%, transparent);
12117+
padding: 0 0 var(--sp-2);
12118+
border-bottom: 1px solid color-mix(in srgb, var(--border) 78%, transparent);
12119+
border-radius: 0;
12120+
background: transparent;
12121+
box-shadow: none;
1212812122
}
1212912123

1213012124
.mobile-files-sheet__segment {
@@ -12133,10 +12127,10 @@ textarea.input {
1213312127
align-items: center;
1213412128
justify-content: flex-start;
1213512129
gap: 6px;
12136-
min-height: 36px;
12137-
padding: 0 4px;
12130+
min-height: 32px;
12131+
padding: 0;
1213812132
border: none;
12139-
border-radius: 999px;
12133+
border-radius: 0;
1214012134
background: transparent;
1214112135
color: var(--text-tertiary);
1214212136
font-size: var(--type-label-size);
@@ -12157,21 +12151,20 @@ textarea.input {
1215712151
.mobile-files-sheet__segment.active::after {
1215812152
content: "";
1215912153
position: absolute;
12160-
right: 4px;
12161-
bottom: 2px;
12162-
left: 4px;
12163-
height: 2px;
12154+
right: 0;
12155+
bottom: -10px;
12156+
left: 0;
12157+
height: 1.5px;
1216412158
border-radius: 999px;
1216512159
background: color-mix(in srgb, var(--accent-blue) 90%, white 10%);
1216612160
}
1216712161

1216812162
.mobile-files-sheet__tab-actions {
1216912163
display: inline-flex;
1217012164
align-items: center;
12171-
gap: 6px;
12165+
gap: 4px;
1217212166
flex-shrink: 0;
1217312167
margin-left: auto;
12174-
padding-bottom: 0;
1217512168
}
1217612169

1217712170
.mobile-files-sheet__tab-action {
@@ -12181,9 +12174,9 @@ textarea.input {
1218112174
width: 32px;
1218212175
height: 32px;
1218312176
padding: 0;
12184-
border: 1px solid color-mix(in srgb, var(--border) 74%, transparent);
12185-
border-radius: 999px;
12186-
background: color-mix(in srgb, var(--bg-surface) 74%, transparent);
12177+
border: none;
12178+
border-radius: 6px;
12179+
background: transparent;
1218712180
color: var(--text-tertiary);
1218812181
transition:
1218912182
background-color var(--duration-fast) var(--ease-out),

packages/web/src/styles/components.theme.test.ts

Lines changed: 45 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1087,6 +1087,17 @@ describe("components.css theme-sensitive surfaces", () => {
10871087
const mobileTerminalSheet = getLastRuleBlock(".mobile-terminal-sheet");
10881088
const mobileTerminal = getLastRuleBlock(".mobile-terminal-sheet .bottom-terminal");
10891089
const mobileFilesSurface = getLastRuleBlock(".mobile-sheet--files .file-tree-shell--mobile");
1090+
const mobileFilesGitSurface = getLastRuleBlock(".mobile-sheet--files .git-panel--mobile");
1091+
const mobileFilesSegmented = getLastRuleBlock(".mobile-files-sheet__segmented");
1092+
const mobileFilesSegment = getLastRuleBlock(".mobile-files-sheet__segment");
1093+
const mobileFilesSegmentActive = getLastRuleBlock(".mobile-files-sheet__segment.active");
1094+
const mobileFilesSegmentIndicator = getLastRuleBlock(
1095+
".mobile-files-sheet__segment.active::after"
1096+
);
1097+
const mobileFilesTabAction = getLastRuleBlock(".mobile-files-sheet__tab-action");
1098+
const mobileFileSearch = getLastRuleBlock(".file-tree-shell--mobile .file-tree-search");
1099+
const mobileFileRow = getLastRuleBlock(".file-tree-shell--mobile .tree-item");
1100+
const mobileFileRowSelected = getLastRuleBlock(".file-tree-shell--mobile .tree-item.selected");
10901101
const supervisorRoot = getLastRuleBlock(".mobile-supervisor-sheet__root");
10911102
const supervisorHeader = getLastRuleBlock(".mobile-supervisor-sheet__detail-header");
10921103
const drawerItem = getLastRuleBlock(".mobile-workspace-drawer__item");
@@ -1104,7 +1115,40 @@ describe("components.css theme-sensitive surfaces", () => {
11041115
expect(mobileTerminalSheet).not.toContain("linear-gradient(");
11051116
expect(mobileTerminal).toContain("border-radius: 0");
11061117
expect(mobileTerminal).not.toContain("var(--radius-xl) var(--radius-xl) 0 0");
1107-
expect(mobileFilesSurface).toContain("border-radius: var(--radius-xl) var(--radius-xl) 0 0");
1118+
expect(mobileFilesSegmented).toContain(
1119+
"border-bottom: 1px solid color-mix(in srgb, var(--border) 78%, transparent)"
1120+
);
1121+
expect(mobileFilesSegmented).toContain("border-radius: 0");
1122+
expect(mobileFilesSegmented).not.toContain("linear-gradient(");
1123+
expect(mobileFilesSegmented).toContain("box-shadow: none");
1124+
expect(mobileFilesSegment).toContain("padding: 0");
1125+
expect(mobileFilesSegment).toContain("font-weight: var(--type-label-weight)");
1126+
expect(mobileFilesSegmentActive).toContain("background: transparent");
1127+
expect(mobileFilesSegmentIndicator).toContain("height: 1.5px");
1128+
expect(mobileFilesTabAction).toContain("border: none");
1129+
expect(mobileFilesTabAction).toContain("border-radius: 6px");
1130+
expect(mobileFilesTabAction).toContain("background: transparent");
1131+
expect(mobileFilesSurface).toContain(
1132+
"border: 1px solid color-mix(in srgb, var(--border) 80%, transparent)"
1133+
);
1134+
expect(mobileFilesSurface).toContain("border-radius: var(--radius-md)");
1135+
expect(mobileFilesSurface).toContain("box-shadow: none");
1136+
expect(mobileFilesSurface).not.toContain("linear-gradient(");
1137+
expect(mobileFilesGitSurface).toContain(
1138+
"border: 1px solid color-mix(in srgb, var(--border) 80%, transparent)"
1139+
);
1140+
expect(mobileFilesGitSurface).toContain("border-radius: var(--radius-md)");
1141+
expect(mobileFilesGitSurface).toContain("box-shadow: none");
1142+
expect(mobileFileSearch).toContain("margin: 0");
1143+
expect(mobileFileSearch).toContain("border-radius: 0");
1144+
expect(mobileFileSearch).toContain("border-right: none");
1145+
expect(mobileFileSearch).toContain("border-left: none");
1146+
expect(mobileFileSearch).toContain("background: transparent");
1147+
expect(mobileFileRow).toContain("min-height: 40px");
1148+
expect(mobileFileRow).toContain("border-radius: 0");
1149+
expect(mobileFileRowSelected).toContain(
1150+
"border-left: 2px solid color-mix(in srgb, var(--accent-blue) 88%, white 12%)"
1151+
);
11081152
expect(supervisorRoot).toContain("border-radius: var(--radius-xl) var(--radius-xl) 0 0");
11091153
expect(supervisorHeader).toContain("border-radius: var(--radius-xl)");
11101154
expect(drawerItem).toContain("border-radius: var(--radius-xl)");

0 commit comments

Comments
 (0)