Skip to content

Commit c768d03

Browse files
committed
Align task step controls with detail tabs
1 parent 902978d commit c768d03

1 file changed

Lines changed: 31 additions & 29 deletions

File tree

components/modals/RepoFormModal.tsx

Lines changed: 31 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1611,14 +1611,37 @@ const TaskStepsEditor: React.FC<{
16111611
</div>
16121612

16131613
<div>
1614-
<nav className="-mb-px flex space-x-4 border-b border-gray-200 dark:border-gray-700">
1615-
<button type="button" onClick={() => setActiveDetailTab('steps')} className={tabButtonClass('steps')}>
1616-
Steps
1617-
</button>
1618-
<button type="button" onClick={() => setActiveDetailTab('variables')} className={tabButtonClass('variables')}>
1619-
Variables
1620-
</button>
1621-
</nav>
1614+
<div className="flex items-end justify-between border-b border-gray-200 dark:border-gray-700">
1615+
<nav className="-mb-px flex space-x-4">
1616+
<button type="button" onClick={() => setActiveDetailTab('steps')} className={tabButtonClass('steps')}>
1617+
Steps
1618+
</button>
1619+
<button type="button" onClick={() => setActiveDetailTab('variables')} className={tabButtonClass('variables')}>
1620+
Variables
1621+
</button>
1622+
</nav>
1623+
1624+
{activeDetailTab === 'steps' && task.steps.length > 0 && (
1625+
<div className="flex items-center gap-2 text-xs pb-2">
1626+
<button
1627+
type="button"
1628+
onClick={handleCollapseAllSteps}
1629+
disabled={collapsibleStepCount === 0}
1630+
className="font-medium text-gray-600 dark:text-gray-300 hover:text-blue-600 disabled:text-gray-400 disabled:cursor-not-allowed"
1631+
>
1632+
Collapse all
1633+
</button>
1634+
<span className="text-gray-300 dark:text-gray-600" aria-hidden="true">|</span>
1635+
<button
1636+
type="button"
1637+
onClick={handleExpandAllSteps}
1638+
className="font-medium text-gray-600 dark:text-gray-300 hover:text-blue-600"
1639+
>
1640+
Expand all
1641+
</button>
1642+
</div>
1643+
)}
1644+
</div>
16221645

16231646
<div className="mt-4 space-y-4">
16241647
{activeDetailTab === 'variables' ? (
@@ -1636,27 +1659,6 @@ const TaskStepsEditor: React.FC<{
16361659
</div>
16371660
)}
16381661

1639-
{task.steps.length > 0 && (
1640-
<div className="flex items-center justify-end gap-2 text-xs">
1641-
<button
1642-
type="button"
1643-
onClick={handleCollapseAllSteps}
1644-
disabled={collapsibleStepCount === 0}
1645-
className="font-medium text-gray-600 dark:text-gray-300 hover:text-blue-600 disabled:text-gray-400 disabled:cursor-not-allowed"
1646-
>
1647-
Collapse all
1648-
</button>
1649-
<span className="text-gray-300 dark:text-gray-600" aria-hidden="true">|</span>
1650-
<button
1651-
type="button"
1652-
onClick={handleExpandAllSteps}
1653-
className="font-medium text-gray-600 dark:text-gray-300 hover:text-blue-600"
1654-
>
1655-
Expand all
1656-
</button>
1657-
</div>
1658-
)}
1659-
16601662
<DockerTaskGenerator dockerCaps={projectInfo?.docker} onAddTask={onAddTask} />
16611663
<NodejsTaskGenerator nodejsCaps={projectInfo?.nodejs} onAddTask={onAddTask} />
16621664
<GoTaskGenerator goCaps={projectInfo?.go} onAddTask={onAddTask} />

0 commit comments

Comments
 (0)