Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion src/sentry/seer/code_review/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,19 @@ def make_seer_request(path: str, payload: Mapping[str, Any]) -> bytes:
Returns:
The response data from the Seer API
"""
# Import here to avoid circular import
from sentry.seer.code_review.webhooks.config import get_direct_to_seer_gh_orgs

repo_owner = payload.get("data", {}).get("repo", {}).get("owner")
direct_to_seer_orgs = get_direct_to_seer_gh_orgs()

seer_url = (
settings.SEER_PREVENT_AI_URL
if (direct_to_seer_orgs and repo_owner and repo_owner in direct_to_seer_orgs)
else settings.SEER_AUTOFIX_URL
)
response = make_signed_seer_api_request(
connection_pool=connection_from_url(settings.SEER_AUTOFIX_URL),
connection_pool=connection_from_url(seer_url),
path=path,
body=orjson.dumps(payload),
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -291,6 +291,12 @@ const StyledTimelineHeader = styled('div')<{isSelected: boolean; isActive?: bool
}

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;
8 changes: 7 additions & 1 deletion static/app/components/events/autofix/autofixTimelineItem.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,13 @@ const StyledTimelineHeader = styled('div')<{isActive?: boolean}>`
}

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,13 @@ const InsightCardRow = styled('div')<{expanded?: boolean; isUserMessage?: boolea
cursor: pointer;

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,12 @@ const SortableThreadSelectorGridCell = styled(ThreadSelectorGridCell)`
user-select: none;
border-radius: ${p => p.theme.radius.md};
&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}
&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down
7 changes: 2 additions & 5 deletions static/app/components/group/groupSummaryWithAutofix.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -356,11 +356,8 @@ const InsightCardButton = styled(motion.div)`
background-color: ${p => p.theme.tokens.background.primary};

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
}

&:active {
opacity: 0.8;
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}
`;

Expand Down
5 changes: 4 additions & 1 deletion static/app/components/prevent/summary.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,10 @@ const SummaryCardContainer = styled(Flex)<{isClickable?: boolean; isFiltered?: b
p.isClickable &&
`
&:hover {
background: ${p.theme.tokens.background.secondary};
background: ${p.theme.tokens.interactive.transparent.neutral.background.hover};
}
&:active {
background: ${p.theme.tokens.interactive.transparent.neutral.background.active};
}
`}
`;
Expand Down
3 changes: 2 additions & 1 deletion static/app/components/scrollCarousel.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,8 @@ const StyledArrowButton = styled(Button)<{orientation: 'horizontal' | 'vertical'

&:hover {
opacity: 1;
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}
`;

Expand Down
10 changes: 9 additions & 1 deletion static/app/components/search/searchResultWrapper.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,15 @@ const SearchResultWrapper = styled('div')<{highlighted?: boolean}>`
p.highlighted &&
css`
color: ${p.theme.activeText};
background: ${p.theme.backgroundSecondary};
background: ${p.theme.tokens.background.secondary};

&:hover {
background: ${p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background: ${p.theme.tokens.interactive.transparent.neutral.background.active};
}
`};

&:not(:first-child) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,8 @@ const AutomationSimpleTableRow = styled(SimpleTable.Row)`
min-height: 54px;

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

@media (hover: hover) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,8 @@ const DetectorSimpleTableRow = styled(SimpleTable.Row)`
min-height: 76px;

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

@media (hover: hover) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,12 @@ const ListItem = styled('li')`
border-bottom: 1px solid ${p => p.theme.tokens.border.primary};
cursor: pointer;
&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}
&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
&:last-child {
border-bottom: 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -259,11 +259,13 @@ const StyledMultipleCheckboxItem = styled(MultipleCheckbox.Item)`
}

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p => p.theme.colors.gray100};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}

& > label {
Expand Down
11 changes: 10 additions & 1 deletion static/app/views/explore/logs/styles.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,18 @@ export const LogTableRow = styled(TableRow)<LogTableRowProps>`
cursor: ${p => (p.isClickable ? 'pointer' : 'default')};

&:hover {
background-color: ${p => p.theme.tokens.background.secondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

${p =>
p.isClickable &&
`
&:active {
background-color: ${p.theme.tokens.interactive.transparent.neutral.background.active};
}
`}

&:not(:last-child) {
border-bottom: 0;
}
Expand Down
16 changes: 14 additions & 2 deletions static/app/views/insights/pages/agents/components/aiSpanList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -426,7 +426,13 @@ const ListItemContainer = styled('div')<{
: 'none'};

&:hover {
background-color: ${p => p.theme.tokens.background.secondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down Expand Up @@ -469,7 +475,13 @@ const TransactionButton = styled('button')`
font-weight: ${p => p.theme.fontWeight.normal};

&:hover:not(:disabled) {
background-color: ${p => p.theme.tokens.background.secondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active:not(:disabled) {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}

&:first-child {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -276,6 +276,12 @@ const InputOutputButton = styled('button')`
width: 100%;

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;
Original file line number Diff line number Diff line change
Expand Up @@ -379,7 +379,10 @@ const MessageBubble = styled('div')<{
cursor: pointer;
&:hover {
border-color: ${p.theme.tokens.border.accent.moderate};
background-color: ${p.theme.tokens.background.secondary};
background-color: ${p.theme.tokens.interactive.transparent.neutral.background.hover};
}
&:active {
background-color: ${p.theme.tokens.interactive.transparent.neutral.background.active};
}
`}
${p =>
Expand Down
2 changes: 1 addition & 1 deletion static/app/views/issueList/pages/topIssuesDrawer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -966,6 +966,6 @@ const TagRow = styled('div')`
}

&:hover {
background: ${p => p.theme.tokens.background.secondary};
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}
`;
24 changes: 21 additions & 3 deletions static/app/views/replays/detail/ai/chapterList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,13 @@ const ChapterWrapper = styled('details')`
/* the border-top is used to eliminate some of the top gap */

&:hover {
border-top: 1px solid ${p => p.theme.tokens.background.secondary};
border-top: 1px solid
${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
border-top: 1px solid
${p => p.theme.tokens.interactive.transparent.neutral.background.active};
}

[data-is-feedback='true'] {
Expand All @@ -314,7 +320,13 @@ const ChapterBreadcrumbRow = styled(BreadcrumbRow)`
}

&:hover {
background-color: ${p => p.theme.tokens.background.secondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand All @@ -327,7 +339,13 @@ const Chapter = styled('summary')`
color: ${p => p.theme.tokens.content.primary};

&:hover {
background-color: ${p => p.theme.backgroundSecondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.active};
}

/* sorry */
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,11 @@ const ToggleButton = styled('button')`
padding: ${space(0.5)} ${space(1)};

:hover {
background: ${p => p.theme.backgroundSecondary};
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}

:active {
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down
2 changes: 1 addition & 1 deletion static/app/views/seerExplorer/askUserQuestionBlock.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ const OptionRow = styled('div')<{isSelected: boolean}>`
transition: background-color 0.15s ease;

&:hover {
background: ${p => p.theme.backgroundSecondary};
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}
`;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -261,8 +261,11 @@ const Suggestion = styled('li')<{active: boolean}>`
font-size: ${p => p.theme.fontSize.md};
cursor: pointer;
background: ${p =>
p.active ? p.theme.backgroundSecondary : p.theme.tokens.background.primary};
p.active ? p.theme.tokens.background.secondary : p.theme.tokens.background.primary};
:hover {
background: ${p => p.theme.backgroundSecondary};
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}
:active {
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;
Original file line number Diff line number Diff line change
Expand Up @@ -491,10 +491,12 @@ const Suggestion = styled('li')<{active: boolean}>`
font-size: ${p => p.theme.fontSize.md};
cursor: pointer;
background: ${p =>
p.active ? p.theme.backgroundSecondary : p.theme.tokens.background.primary};
p.active ? p.theme.tokens.background.secondary : p.theme.tokens.background.primary};
:hover {
background: ${p =>
p.active ? p.theme.backgroundSecondary : p.theme.backgroundSecondary};
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}
:active {
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down
3 changes: 2 additions & 1 deletion static/app/views/settings/components/dataScrubbing/rules.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,8 @@ const ListItem = styled('li')`
padding: ${space(1)} ${space(2)};
border-bottom: 1px solid ${p => p.theme.tokens.border.primary};
&:hover {
background-color: ${p => p.theme.tokens.background.secondary};
background-color: ${p =>
p.theme.tokens.interactive.transparent.neutral.background.hover};
}
&:last-child {
border-bottom: 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,11 @@ const ItemHeader = styled('button')`
text-align: left;

&:hover {
background: ${p => p.theme.backgroundSecondary};
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.hover};
}

&:active {
background: ${p => p.theme.tokens.interactive.transparent.neutral.background.active};
}
`;

Expand Down
Loading
Loading