mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-09-21 02:33:32 +00:00
Remove duplicate Patrol findings card header
Remove the redundant Patrol findings card header so the selected tab remains the sole owner of findings-surface framing and count context.
This commit is contained in:
@@ -1074,35 +1074,8 @@ export const FindingsPanel: Component<FindingsPanelProps> = (props) => {
|
||||
</div>
|
||||
</Show>
|
||||
|
||||
{/* Patrol findings section */}
|
||||
<Show when={!aiIntelligenceStore.findingsLoading}>
|
||||
<Card padding="none" class="overflow-hidden">
|
||||
{/* Header */}
|
||||
<div class="bg-blue-50 dark:bg-blue-900 px-4 py-3 border-b border-border">
|
||||
<div class="flex items-center justify-between">
|
||||
<div class="flex items-center gap-2">
|
||||
<svg
|
||||
class="w-5 h-5 text-blue-600 dark:text-blue-400"
|
||||
fill="none"
|
||||
stroke="currentColor"
|
||||
viewBox="0 0 24 24"
|
||||
>
|
||||
<path
|
||||
stroke-linecap="round"
|
||||
stroke-linejoin="round"
|
||||
stroke-width="2"
|
||||
d="M9.663 17h4.673M12 3v1m6.364 1.636l-.707.707M21 12h-1M4 12H3m3.343-5.657l-.707-.707m2.828 9.9a5 5 0 117.072 0l-.548.547A3.374 3.374 0 0014 18.469V19a2 2 0 11-4 0v-.531c0-.895-.356-1.754-.988-2.386l-.548-.547z"
|
||||
/>
|
||||
</svg>
|
||||
<span class="font-medium text-base-content">Patrol findings</span>
|
||||
<Show when={patrolFindings().length > 0}>
|
||||
<span class="px-2 py-0.5 text-xs font-medium bg-blue-200 dark:bg-blue-700 text-blue-800 dark:text-blue-200 rounded-full">
|
||||
{patrolFindings().length}
|
||||
</span>
|
||||
</Show>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
{/* Content */}
|
||||
<div class="divide-y divide-border-subtle">
|
||||
<Show when={patrolFindings().length === 0}>
|
||||
|
||||
@@ -167,7 +167,9 @@ describe('aiFindingPresentation', () => {
|
||||
});
|
||||
|
||||
it('keeps the findings card header functional instead of repeating product marketing copy', () => {
|
||||
expect(findingsPanelSource).toContain('Patrol findings');
|
||||
expect(findingsPanelSource).not.toContain(
|
||||
'<span class="font-medium text-base-content">Patrol findings</span>',
|
||||
);
|
||||
expect(findingsPanelSource).not.toContain('Pulse Patrol Findings');
|
||||
expect(findingsPanelSource).not.toContain('AI-discovered insights');
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user