Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit dc1da00

Browse files
committed
Fix active style
1 parent 2a78a9c commit dc1da00

File tree

3 files changed

+41
-41
lines changed

3 files changed

+41
-41
lines changed

client/web/src/enterprise/codeintel/configuration/components/RepositoryPatternList.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@ import {
1111
ErrorAlert,
1212
Icon,
1313
Input,
14-
InputErrorMessage,
15-
InputStatus,
1614
Link,
1715
LoadingSpinner,
1816
Text,

client/web/src/enterprise/codeintel/configuration/pages/CodeIntelConfigurationPage.module.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,4 +48,8 @@
4848
&:not(:last-child) {
4949
border-bottom: 1px solid var(--border-color);
5050
}
51+
52+
&[data-reach-menu-item]:active {
53+
--text-muted: var(--white);
54+
}
5155
}

client/web/src/enterprise/codeintel/configuration/pages/CodeIntelConfigurationPage.tsx

Lines changed: 37 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -204,49 +204,47 @@ interface CreatePolicyButtonsProps {
204204
repo?: { id: string; name: string }
205205
}
206206

207-
const CreatePolicyButtons: FunctionComponent<CreatePolicyButtonsProps> = ({ repo, history }) => (
208-
<>
209-
<Menu>
210-
<ButtonGroup>
211-
<Button to="./configuration/new?type=head" variant="primary" as={Link}>
212-
Create new {!repo && 'global'} policy
213-
</Button>
214-
<MenuButton variant="primary" className={styles.dropdownButton}>
215-
<Icon aria-hidden={true} svgPath={mdiChevronDown} />
216-
<VisuallyHidden>Actions</VisuallyHidden>
217-
</MenuButton>
218-
</ButtonGroup>
219-
<MenuList position={Position.bottomEnd} className={styles.dropdownList}>
220-
<MenuLink as={Link} className={styles.dropdownItem} to="./configuration/new?type=head">
221-
<>
222-
<Text weight="medium" className="mb-2">
223-
Create new {!repo && 'global'} policy for HEAD
224-
</Text>
225-
<Text className="mb-0 text-muted">
226-
Match the tip of the default branch{' '}
227-
{repo ? 'within this repository' : 'across multiple repositories'}
228-
</Text>
229-
</>
230-
</MenuLink>
231-
<MenuLink as={Link} className={styles.dropdownItem} to="./configuration/new?type=branch">
232-
<Text weight="medium" className="mb-2">
233-
Create new {!repo && 'global'} branch policy
234-
</Text>
235-
<Text className="mb-0 text-muted">
236-
Match multiple branches {repo ? 'within this repository' : 'across multiple repositories'}
237-
</Text>
238-
</MenuLink>
239-
<MenuLink as={Link} className={styles.dropdownItem} to="./configuration/new?type=tag">
207+
const CreatePolicyButtons: FunctionComponent<CreatePolicyButtonsProps> = ({ repo }) => (
208+
<Menu>
209+
<ButtonGroup>
210+
<Button to="./configuration/new?type=head" variant="primary" as={Link}>
211+
Create new {!repo && 'global'} policy
212+
</Button>
213+
<MenuButton variant="primary" className={styles.dropdownButton}>
214+
<Icon aria-hidden={true} svgPath={mdiChevronDown} />
215+
<VisuallyHidden>Actions</VisuallyHidden>
216+
</MenuButton>
217+
</ButtonGroup>
218+
<MenuList position={Position.bottomEnd} className={styles.dropdownList}>
219+
<MenuLink as={Link} className={styles.dropdownItem} to="./configuration/new?type=head">
220+
<>
240221
<Text weight="medium" className="mb-2">
241-
Create new {!repo && 'global'} tag policy
222+
Create new {!repo && 'global'} policy for HEAD
242223
</Text>
243224
<Text className="mb-0 text-muted">
244-
Match multiple tags {repo ? 'within this repository' : 'across multiple repositories'}
225+
Match the tip of the default branch{' '}
226+
{repo ? 'within this repository' : 'across multiple repositories'}
245227
</Text>
246-
</MenuLink>
247-
</MenuList>
248-
</Menu>
249-
</>
228+
</>
229+
</MenuLink>
230+
<MenuLink as={Link} className={styles.dropdownItem} to="./configuration/new?type=branch">
231+
<Text weight="medium" className="mb-2">
232+
Create new {!repo && 'global'} branch policy
233+
</Text>
234+
<Text className="mb-0 text-muted">
235+
Match multiple branches {repo ? 'within this repository' : 'across multiple repositories'}
236+
</Text>
237+
</MenuLink>
238+
<MenuLink as={Link} className={styles.dropdownItem} to="./configuration/new?type=tag">
239+
<Text weight="medium" className="mb-2">
240+
Create new {!repo && 'global'} tag policy
241+
</Text>
242+
<Text className="mb-0 text-muted">
243+
Match multiple tags {repo ? 'within this repository' : 'across multiple repositories'}
244+
</Text>
245+
</MenuLink>
246+
</MenuList>
247+
</Menu>
250248
)
251249

252250
interface PoliciesNodeProps {

0 commit comments

Comments
 (0)