Skip to content

Commit ac80a89

Browse files
fixes
1 parent e746b89 commit ac80a89

File tree

5 files changed

+18
-32
lines changed

5 files changed

+18
-32
lines changed

packages/ui/locales/en/component.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,12 @@
5353
"remove": "Remove",
5454
"reorder": "Reorder",
5555
"unpin": "Unpin",
56+
"appearence": "Appearance",
57+
"language": "Language",
58+
"logout": "Log out",
5659
"profile": "Profile",
5760
"theme": "Theme",
5861
"customNav": "Customize navigation",
59-
"logout": "Log out",
6062
"devops": "Devops",
6163
"repositories": "Repositories",
6264
"pipelines": "Pipelines",

packages/ui/locales/fr/component.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,12 @@
5353
"remove": "Supprimer",
5454
"reorder": "Réorganiser",
5555
"unpin": "Désépingler",
56+
"appearence": "Appearance",
57+
"language": "Language",
58+
"logout": "Se déconnecter",
5659
"profile": "Profile",
5760
"theme": "Thème",
5861
"customNav": "Personnaliser la navigation",
59-
"logout": "Se déconnecter",
6062
"devops": "Devops",
6163
"repositories": "Dépôts",
6264
"pipelines": "Pipelines",

packages/ui/locales/fr/views.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -500,4 +500,4 @@
500500
"secrets": {
501501
"secretsTitle": "Secrets"
502502
}
503-
}
503+
}

packages/ui/src/views/layouts/PullRequestLayout.tsx

Lines changed: 10 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
import { FC, PropsWithChildren, useCallback } from 'react'
2-
import { useLocation, useNavigate } from 'react-router-dom'
1+
import { FC, PropsWithChildren } from 'react'
32

43
import { Badge, BadgeProps, Icon, IconProps, Tabs, TabsTriggerProps } from '@/components'
54
import { useRouterContext } from '@/context'
@@ -43,41 +42,25 @@ export const PullRequestLayout: FC<PullRequestLayoutProps> = ({
4342
repoId,
4443
updateTitle
4544
}) => {
46-
const { Outlet } = useRouterContext()
45+
const { Outlet, navigate } = useRouterContext()
4746
const { pullRequest } = usePullRequestStore()
4847
const { t } = useTranslationStore()
49-
const navigate = useNavigate()
50-
const location = useLocation()
5148

52-
const getIsActiveTab = useCallback(
53-
(tab: string) => (location.pathname.endsWith(tab) ? 'active' : 'inactive'),
54-
[location.pathname]
55-
)
56-
const makeHandleTabChange = useCallback((tab: string) => () => navigate(tab), [navigate])
57-
58-
const getTabProps = useCallback(
59-
(tab: PullRequestTabsKeys): TabsTriggerProps => ({
60-
value: tab,
61-
...{ 'data-state': getIsActiveTab(tab) },
62-
onClick: makeHandleTabChange(tab),
63-
className: 'group gap-x-1.5',
64-
role: 'link'
65-
}),
66-
[getIsActiveTab, makeHandleTabChange]
67-
)
49+
const getTabProps = (tab: PullRequestTabsKeys): TabsTriggerProps => ({
50+
value: tab,
51+
onClick: () => navigate(tab),
52+
className: 'group gap-x-1.5',
53+
role: 'link'
54+
})
6855

6956
return (
7057
<SandboxLayout.Main fullWidth>
7158
<SandboxLayout.Content className="mx-auto max-w-[1500px] px-6">
7259
{pullRequest && (
73-
<PullRequestHeader
74-
className="mb-10"
75-
updateTitle={updateTitle}
76-
data={{ ...pullRequest, stats: { commits: pullRequest?.stats?.commits }, spaceId, repoId }}
77-
/>
60+
<PullRequestHeader className="mb-10" updateTitle={updateTitle} data={{ ...pullRequest, spaceId, repoId }} />
7861
)}
7962

80-
<Tabs.Root variant="tabnav" className="mb-7">
63+
<Tabs.Root variant="tabnav" className="mb-7" defaultValue={PullRequestTabsKeys.CONVERSATION}>
8164
<Tabs.List className="before:left-1/2 before:w-[calc(100vw-220px)] before:-translate-x-1/2">
8265
<Tabs.Trigger {...getTabProps(PullRequestTabsKeys.CONVERSATION)}>
8366
<TabTitleWithIcon icon="comments">

packages/ui/src/views/repo/pull-request/pull-request-list-page.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,8 +198,7 @@ const PullRequestListPage: FC<PullRequestPageProps> = ({
198198
<>
199199
<PRListFilterHandler
200200
ref={filtersRef}
201-
// TODO: fix type issue
202-
{...{ onFilterSelectionChange: onFilterSelectionChange }}
201+
onFilterSelectionChange={onFilterSelectionChange}
203202
onChange={onFilterValueChange}
204203
view="dropdown"
205204
>

0 commit comments

Comments
 (0)