refactor(task-list): adjust subtask indentation for improved visual hierarchy
- Reduced spacing for level 1 subtasks and increased spacing for deeper levels in TaskRowWithSubtasks and TitleColumn components. - Enhanced comments to clarify indentation logic for better maintainability.
This commit is contained in:
@@ -139,13 +139,13 @@ const AddSubtaskRow: React.FC<AddSubtaskRowProps> = memo(({
|
|||||||
return (
|
return (
|
||||||
<div className="flex items-center h-full" style={baseStyle}>
|
<div className="flex items-center h-full" style={baseStyle}>
|
||||||
<div className="flex items-center w-full h-full">
|
<div className="flex items-center w-full h-full">
|
||||||
{/* Match subtask indentation pattern - tighter spacing */}
|
{/* Match subtask indentation pattern - reduced spacing for level 1 */}
|
||||||
<div className="w-3" />
|
|
||||||
{/* Add additional indentation for deeper levels - 16px per level */}
|
|
||||||
{Array.from({ length: depth }).map((_, i) => (
|
|
||||||
<div key={i} className="w-4" />
|
|
||||||
))}
|
|
||||||
<div className="w-2" />
|
<div className="w-2" />
|
||||||
|
{/* Add additional indentation for deeper levels - increased spacing for level 2+ */}
|
||||||
|
{Array.from({ length: depth }).map((_, i) => (
|
||||||
|
<div key={i} className="w-6" />
|
||||||
|
))}
|
||||||
|
<div className="w-1" />
|
||||||
|
|
||||||
{isActive ? (
|
{isActive ? (
|
||||||
!isAdding ? (
|
!isAdding ? (
|
||||||
|
|||||||
@@ -152,12 +152,12 @@ export const TitleColumn: React.FC<TitleColumnProps> = memo(({
|
|||||||
/* Normal layout when not editing */
|
/* Normal layout when not editing */
|
||||||
<>
|
<>
|
||||||
<div className="flex items-center flex-1 min-w-0">
|
<div className="flex items-center flex-1 min-w-0">
|
||||||
{/* Indentation for subtasks - tighter spacing */}
|
{/* Indentation for subtasks - reduced spacing for level 1 */}
|
||||||
{isSubtask && <div className="w-3 flex-shrink-0" />}
|
{isSubtask && <div className="w-2 flex-shrink-0" />}
|
||||||
|
|
||||||
{/* Additional indentation for deeper levels - 16px per level */}
|
{/* Additional indentation for deeper levels - increased spacing for level 2+ */}
|
||||||
{Array.from({ length: depth }).map((_, i) => (
|
{Array.from({ length: depth }).map((_, i) => (
|
||||||
<div key={i} className="w-4 flex-shrink-0" />
|
<div key={i} className="w-6 flex-shrink-0" />
|
||||||
))}
|
))}
|
||||||
|
|
||||||
{/* Expand/Collapse button - show for any task that can have sub-tasks */}
|
{/* Expand/Collapse button - show for any task that can have sub-tasks */}
|
||||||
@@ -182,8 +182,8 @@ export const TitleColumn: React.FC<TitleColumnProps> = memo(({
|
|||||||
</button>
|
</button>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
{/* Additional indentation for subtasks after the expand button space */}
|
{/* Additional indentation for subtasks after the expand button space - reduced for level 1 */}
|
||||||
{isSubtask && <div className="w-2 flex-shrink-0" />}
|
{isSubtask && <div className="w-1 flex-shrink-0" />}
|
||||||
|
|
||||||
<div className="flex items-center gap-2 flex-1 min-w-0">
|
<div className="flex items-center gap-2 flex-1 min-w-0">
|
||||||
{/* Task name with dynamic width */}
|
{/* Task name with dynamic width */}
|
||||||
|
|||||||
Reference in New Issue
Block a user