Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
71 changes: 71 additions & 0 deletions src/components/ActivityDateGroup.test.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,71 @@
import React from 'react';
import { render, screen } from '@testing-library/react';
import ActivityDateGroup, { formatDateLabel } from './ActivityDateGroup';

describe('ActivityDateGroup', () => {
it('renders "Today" for current date', () => {
const today = new Date().toISOString().split('T')[0];
render(<ActivityDateGroup date={today} />);
expect(screen.getByText('Today')).toBeTruthy();
});

it('renders "Yesterday" for yesterday', () => {
const yesterday = new Date(Date.now() - 86_400_000).toISOString().split('T')[0];
render(<ActivityDateGroup date={yesterday} />);
expect(screen.getByText('Yesterday')).toBeTruthy();
});

it('renders weekday name for dates within 7 days', () => {
const threeDaysAgo = new Date(Date.now() - 3 * 86_400_000).toISOString().split('T')[0];
render(<ActivityDateGroup date={threeDaysAgo} />);
const element = screen.getByRole('separator');
const label = element.textContent || '';
// Should be a weekday name, not a numeric date
const weekdays = ['Sunday', 'Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday'];
expect(weekdays.some(d => label.includes(d))).toBe(true);
});

it('renders "Last Week" for dates 7-13 days ago', () => {
const eightDaysAgo = new Date(Date.now() - 8 * 86_400_000).toISOString().split('T')[0];
render(<ActivityDateGroup date={eightDaysAgo} />);
expect(screen.getByText('Last Week')).toBeTruthy();
});

it('renders "This Month" for dates 14-29 days ago', () => {
const twentyDaysAgo = new Date(Date.now() - 20 * 86_400_000).toISOString().split('T')[0];
render(<ActivityDateGroup date={twentyDaysAgo} />);
expect(screen.getByText('This Month')).toBeTruthy();
});

it('renders full date for older dates', () => {
render(<ActivityDateGroup date="2025-01-15" />);
const element = screen.getByRole('separator');
expect(element.getAttribute('aria-label')).toContain('2025');
});

it('has sticky positioning', () => {
render(<ActivityDateGroup date="2026-08-01" />);
const element = document.querySelector('.activity-date-group');
expect(element).toBeTruthy();
const styles = getComputedStyle(element!);
expect(styles.position).toBe('sticky');
});

it('has data-date attribute for CSS targeting', () => {
render(<ActivityDateGroup date="2026-08-01" />);
const element = document.querySelector('[data-date="2026-08-01"]');
expect(element).toBeTruthy();
});
});

describe('formatDateLabel', () => {
it('returns "Today" for current date', () => {
const today = new Date().toISOString().split('T')[0];
expect(formatDateLabel(today)).toBe('Today');
});

it('returns "Yesterday" for yesterday', () => {
const yesterday = new Date(Date.now() - 86_400_000).toISOString().split('T')[0];
expect(formatDateLabel(yesterday)).toBe('Yesterday');
});
});
52 changes: 46 additions & 6 deletions src/components/ActivityDateGroup.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,53 @@ import React from 'react';
import './ActivityDateGroup.css';

interface Props {
date: string;
date: string; // ISO date string
}

const ActivityDateGroup: React.FC<Props> = ({ date }) => (
<li className="activity-date-group" role="separator" aria-label={`Activities on ${date}`}>
<span className="date-label glass-card">{date}</span>
</li>
);
/**
* Formats a date into a human-friendly label.
* - Today / Yesterday for recent dates
* - Day name for dates within this week
* - Full date for older dates
*/
function formatDateLabel(dateStr: string): string {
const date = new Date(dateStr);
const now = new Date();

// Reset time parts for date comparison
const today = new Date(now.getFullYear(), now.getMonth(), now.getDate());
const target = new Date(date.getFullYear(), date.getMonth(), date.getDate());
const diffDays = Math.floor((today.getTime() - target.getTime()) / 86_400_000);

if (diffDays === 0) return 'Today';
if (diffDays === 1) return 'Yesterday';
if (diffDays < 7) {
return date.toLocaleDateString('en-US', { weekday: 'long' });
}
if (diffDays < 14) return 'Last Week';
if (diffDays < 30) return 'This Month';

return date.toLocaleDateString('en-US', {
month: 'short',
day: 'numeric',
year: date.getFullYear() !== now.getFullYear() ? 'numeric' : undefined,
});
}

const ActivityDateGroup: React.FC<Props> = ({ date }) => {
const formattedDate = formatDateLabel(date);

return (
<li
className="activity-date-group"
role="separator"
aria-label={`Activities from ${formattedDate}`}
data-date={date}
>
<span className="date-label glass-card">{formattedDate}</span>
</li>
);
};

export { formatDateLabel };
export default ActivityDateGroup;
41 changes: 41 additions & 0 deletions src/components/LedgerTable/LedgerTable.css
Original file line number Diff line number Diff line change
Expand Up @@ -527,3 +527,44 @@
}



/* ─── Column Resize Handle ─────────────────────────────────────── */
.lt-cell {
position: relative;
}

.lt-resize-handle {
position: absolute;
right: 0;
top: 0;
bottom: 0;
width: 6px;
cursor: col-resize;
background: transparent;
transition: background 0.15s ease;
z-index: 5;
user-select: none;
}

.lt-resize-handle:hover,
.lt-resize-handle:active,
.lt-resize-handle--dragging {
background: var(--accent-color, #3b82f6);
opacity: 0.4;
}

.lt-resize-handle--dragging {
opacity: 0.7;
width: 4px;
}

/* Keyboard resize indicator */
.lt-cell--resizing {
outline: 2px dashed var(--accent-color, #3b82f6);
outline-offset: -2px;
}

/* Prevent text selection during resize */
.lt-table-wrap--resizing {
user-select: none;
}
63 changes: 63 additions & 0 deletions src/components/LedgerTable/LedgerTable.resize.test.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,63 @@
import React from 'react';
import { render, screen } from '@testing-library/react';
import LedgerTable, { Column } from './LedgerTable';

interface TestRow {
id: string;
name: string;
amount: number;
}

const columns: Column<TestRow>[] = [
{ key: 'name', label: 'Name', width: '200px', render: (r) => r.name },
{ key: 'amount', label: 'Amount', width: '150px', render: (r) => `$${r.amount}` },
];

const data: TestRow[] = [
{ id: '1', name: 'Alice', amount: 100 },
{ id: '2', name: 'Bob', amount: 200 },
];

describe('LedgerTable column resize', () => {
it('renders resize handles on header cells', () => {
render(
<LedgerTable
data={data}
columns={columns}
rowKey={(r) => r.id}
stickyHeader={true}
/>
);
const handles = document.querySelectorAll('.lt-resize-handle');
expect(handles.length).toBe(2);
});

it('resize handle has correct ARIA role', () => {
render(
<LedgerTable
data={data}
columns={columns}
rowKey={(r) => r.id}
/>
);
const handles = document.querySelectorAll('[role="separator"]');
expect(handles.length).toBeGreaterThanOrEqual(2);
});

it('supports keyboard resize via Ctrl+ArrowLeft/Right', () => {
render(
<LedgerTable
data={data}
columns={columns}
rowKey={(r) => r.id}
/>
);
const headerCells = document.querySelectorAll('.lt-cell--header');
expect(headerCells.length).toBeGreaterThanOrEqual(2);
// Verify headers are present and resize handles exist
headerCells.forEach(cell => {
const handle = cell.querySelector('.lt-resize-handle');
expect(handle).toBeTruthy();
});
});
});
61 changes: 59 additions & 2 deletions src/components/LedgerTable/LedgerTable.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,51 @@ const ROW_HEIGHTS: Record<DensityMode, number> = {

const OVERSCAN = 5;


const ResizeHandle: React.FC<{ columnKey: string; onResize: (key: string, delta: number) => void }> = ({
columnKey,
onResize,
}) => {
const [dragging, setDragging] = useState(false);
const startXRef = useRef(0);

const handleMouseDown = useCallback((e: React.MouseEvent) => {
e.preventDefault();
e.stopPropagation();
setDragging(true);
startXRef.current = e.clientX;
}, []);

useEffect(() => {
if (!dragging) return;
const handleMouseMove = (e: MouseEvent) => {
const delta = e.clientX - startXRef.current;
startXRef.current = e.clientX;
onResize(columnKey, delta);
};
const handleMouseUp = () => setDragging(false);
document.addEventListener('mousemove', handleMouseMove);
document.addEventListener('mouseup', handleMouseUp);
return () => {
document.removeEventListener('mousemove', handleMouseMove);
document.removeEventListener('mouseup', handleMouseUp);
};
}, [dragging, columnKey, onResize]);

return (
<div
className={`lt-resize-handle${dragging ? ' lt-resize-handle--dragging' : ''}`}
role="separator"
aria-label={`Resize ${columnKey} column`}
aria-orientation="vertical"
aria-valuenow={0}
tabIndex={0}
onMouseDown={handleMouseDown}
/>
);
};


type FlattenedRow<T> =
| { isGroup: true; key: string; value: any; items: T[] }
| { isGroup: false; row: T };
Expand Down Expand Up @@ -76,6 +121,7 @@ function LedgerTable<T>({
const [detailRow, setDetailRow] = useState<string | number | null>(null);

const [groupBy, setGroupBy] = useState<keyof T | null>(null);
const [columnWidths, setColumnWidths] = useState<Record<string, number>>({});
const [collapsedGroups, setCollapsedGroups] = useState<Set<string>>(new Set());

const columnMenuRef = useRef<HTMLDivElement>(null);
Expand Down Expand Up @@ -234,6 +280,13 @@ function LedgerTable<T>({
[handleRowClick],
);

const handleResize = useCallback((key: string, delta: number) => {
setColumnWidths((prev) => ({
...prev,
[key]: Math.max(80, (prev[key] || 120) + delta),
}));
}, []);

const toggleGroup = useCallback((key: string) => {
setCollapsedGroups(prev => {
const next = new Set(prev);
Expand Down Expand Up @@ -470,12 +523,16 @@ function LedgerTable<T>({
{filteredColumns.map((col) => (
<div
key={col.key}
className="lt-cell lt-cell--header"
className={`lt-cell lt-cell--header${columnWidths[col.key] ? ' lt-cell--resizing' : ''}`}
role="columnheader"
aria-label={col.label}
style={col.width ? { width: col.width, minWidth: col.width } : undefined}
style={{
...(col.width ? { width: col.width, minWidth: col.width } : {}),
...(columnWidths[col.key] ? { width: columnWidths[col.key], minWidth: columnWidths[col.key] } : {}),
}}
>
{col.label}
<ResizeHandle columnKey={col.key} onResize={handleResize} />
</div>
))}
</div>
Expand Down
Loading