root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(FN-5619): expand PR notice block padding

Browse Source 
Adds padding expansion to the PR notice block in PrPanel.css with corresponding test assertions to verify the token values, completing the FN-5619 styling pass.

Fusion-Task-Id: FN-5619

Co-authored-by: Fusion (runfusion.ai) <noreply@runfusion.ai>
Fusion-Task-Id: FN-5619
This commit is contained in:
gsxdsm
2026-05-28 08:44:29 -07:00
parent 0a04837e7a
commit 9969b0779b
2 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
packages/dashboard/app/components/PrPanel.css
View File

@@ -179,7 +179,7 @@
border-radius: var(--radius-md);
color: var(--color-warning);
font-size: 0.8125rem;
padding: var(--space-sm);
padding: var(--space-md);
}
.pr-hint--conflict {
@@ -193,7 +193,7 @@
font-size: 0.8125rem;
gap: var(--space-sm);
justify-content: space-between;
padding: var(--space-sm);
padding: var(--space-md);
}
.pr-hint--success {
@@ -202,7 +202,7 @@
border-radius: var(--radius-md);
color: var(--color-success);
font-size: 0.8125rem;
padding: var(--space-sm);
padding: var(--space-md);
}
.pr-merge-controls {
@@ -216,7 +216,7 @@
border-radius: var(--radius-md);
display: grid;
gap: var(--space-sm);
padding: var(--space-sm);
padding: var(--space-md);
}
.pr-conflict-section__header {
@@ -249,7 +249,7 @@
display: flex;
gap: var(--space-sm);
justify-content: space-between;
padding: var(--space-sm);
padding: var(--space-md);
}
.pr-error {
@@ -259,7 +259,7 @@
color: var(--color-error);
display: grid;
gap: var(--space-sm);
padding: var(--space-sm);
padding: var(--space-md);
}
.pr-error__hint {

6
packages/dashboard/app/components/__tests__/TaskDetailModal.attachments-and-tabs.test.tsx
View File

@@ -331,6 +331,12 @@ describe("TaskDetailModal", () => {
const css = readDashboardStylesSource();
expectBaseRule(css, ".detail-pr-tab", "gap: var(--space-lg);");
expectBaseRule(css, ".detail-pr-tab .detail-section,\n.detail-pr-section", "margin-top: 0;");
expectBaseRule(css, ".pr-hint--warning", "padding: var(--space-md);");
expectBaseRule(css, ".pr-hint--conflict", "padding: var(--space-md);");
expectBaseRule(css, ".pr-hint--success", "padding: var(--space-md);");
expectBaseRule(css, ".pr-conflict-section", "padding: var(--space-md);");
expectBaseRule(css, ".pr-merge-error", "padding: var(--space-md);");
expectBaseRule(css, ".pr-error", "padding: var(--space-md);");
});
it("activity list does not have nested scroll constraints", () => {