fix(#92): use PDA-friendly language (Target instead of Due)
Critical PDA-friendly design compliance fix.
Changed forbidden "Due:" to approved "Target:" throughout FederatedTaskCard
component and tests, per DESIGN-PRINCIPLES.md requirements.
Changes:
- FederatedTaskCard.tsx: Changed "Due: {dueDate}" to "Target: {dueDate}"
- FederatedTaskCard.test.tsx: Updated all test expectations from "Due:" to "Target:"
- Updated test names to reflect "target date" terminology
All 11 tests passing.
This ensures full compliance with PDA-friendly language guidelines:
| ❌ NEVER | ✅ ALWAYS |
| DUE | Target date |
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -66,11 +66,11 @@ describe("FederatedTaskCard", () => {
|
|||||||
expect(screen.getByText("High")).toBeInTheDocument();
|
expect(screen.getByText("High")).toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("should render due date", () => {
|
it("should render target date", () => {
|
||||||
render(<FederatedTaskCard federatedTask={mockTask} />);
|
render(<FederatedTaskCard federatedTask={mockTask} />);
|
||||||
|
|
||||||
// Check for "Due:" text followed by a date
|
// Check for "Target:" text followed by a date
|
||||||
expect(screen.getByText(/Due:/)).toBeInTheDocument();
|
expect(screen.getByText(/Target:/)).toBeInTheDocument();
|
||||||
expect(screen.getByText(/2026/)).toBeInTheDocument();
|
expect(screen.getByText(/2026/)).toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -113,8 +113,8 @@ describe("FederatedTaskCard", () => {
|
|||||||
).not.toBeInTheDocument();
|
).not.toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("should handle task without due date", () => {
|
it("should handle task without target date", () => {
|
||||||
const taskNoDue: FederatedTask = {
|
const taskNoTarget: FederatedTask = {
|
||||||
...mockTask,
|
...mockTask,
|
||||||
task: {
|
task: {
|
||||||
...mockTask.task,
|
...mockTask.task,
|
||||||
@@ -122,10 +122,10 @@ describe("FederatedTaskCard", () => {
|
|||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
render(<FederatedTaskCard federatedTask={taskNoDue} />);
|
render(<FederatedTaskCard federatedTask={taskNoTarget} />);
|
||||||
|
|
||||||
expect(screen.getByText("Review pull request")).toBeInTheDocument();
|
expect(screen.getByText("Review pull request")).toBeInTheDocument();
|
||||||
expect(screen.queryByText(/Due:/)).not.toBeInTheDocument();
|
expect(screen.queryByText(/Target:/)).not.toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("should use PDA-friendly language for status", () => {
|
it("should use PDA-friendly language for status", () => {
|
||||||
|
|||||||
@@ -105,8 +105,8 @@ export function FederatedTaskCard({
|
|||||||
{/* Priority */}
|
{/* Priority */}
|
||||||
<span className={`text-xs font-medium ${priority.colorClass}`}>{priority.text}</span>
|
<span className={`text-xs font-medium ${priority.colorClass}`}>{priority.text}</span>
|
||||||
|
|
||||||
{/* Due date */}
|
{/* Target date */}
|
||||||
{dueDate && <span className="text-xs text-gray-600">Due: {dueDate}</span>}
|
{dueDate && <span className="text-xs text-gray-600">Target: {dueDate}</span>}
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
|
|||||||
Reference in New Issue
Block a user