Skip to content

[pull] master from getsentry:master#1810

Merged
pull[bot] merged 10 commits intoKingDEV95:masterfrom
getsentry:master
Mar 20, 2026
Merged

[pull] master from getsentry:master#1810
pull[bot] merged 10 commits intoKingDEV95:masterfrom
getsentry:master

Conversation

@pull
Copy link

@pull pull bot commented Mar 20, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

jonathanhefner and others added 10 commits March 19, 2026 15:30
…e paths (#111048)

`${CLAUDE_SKILL_ROOT}` is not a recognized Claude Code variable, so all
22 file references using it were passed through as literal strings and
never resolved. While `${CLAUDE_SKILL_DIR}` does work, it is scoped to
bash injection commands (backtick-bang syntax) where the working
directory may differ from the skill directory. For plain markdown file
references like these, bare relative paths (e.g.
`references/endpoint-patterns.md`) are the standard approach per the
Agent Skills spec.

---------

Co-authored-by: David Cramer <dcramer@gmail.com>
Co-authored-by: Codex <noreply@openai.com>
We want to support agent handoffs earlier in the process.
Fixes an issue with secondary navigation where page scroll was triggered
during keyboard navigation

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Swaps the navigation to a moderate enter animation which looks less
springy

---------

Co-authored-by: Nate Moore <nate.moore@sentry.io>
Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>
<!-- CURSOR_AGENT_PR_BODY_BEGIN -->
This PR improves the UX of the `IssueViewQueryCount` component by
removing the fixed minimum width and adding a smooth entrance animation.

## Problem
The fixed `min-width: 4ch` was causing issue view titles to be truncated
when the count numbers were displayed. While this prevented layout shift
when numbers loaded, it resulted in a worse UX by cutting off view
titles, particularly when longer count numbers (like "99+") were
displayed.

## Solution
1. **Removed `min-width: 4ch` constraint** - Allows the tag component to
size naturally based on its content
2. **Hide tag until data loads** - Only render the tag once data has
been fetched, eliminating layout shift
3. **Added entrance animation** - Tag fades in with opacity (0 → 1) and
scales up (0.95 → 1) over 300ms for a polished appearance
4. **Maintained refetch animation** - Existing background color pulse
animation still works during data refetches

This eliminates both the layout shift and title truncation issues,
providing a cleaner, more natural appearance.

### Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated
in the State of Delaware in 2015 as Functional Software, Inc. and is
gonna need some rights from me in order to utilize my contributions in
this here PR. So here's the deal: I retain all rights, title and
interest in and to my contributions, and by keeping this boilerplate
intact I confirm that Sentry can use, modify, copy, and redistribute my
contributions, under Sentry's choice of terms.
<!-- CURSOR_AGENT_PR_BODY_END -->

[Slack
Thread](https://sentry.slack.com/archives/C08QLT0PYQK/p1773957429531369?thread_ts=1773957429.531369&cid=C08QLT0PYQK)

<div><a
href="https://cursor.com/agents/bc-8f58544c-4504-5696-b018-da550b0b8144"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-web-light.png"><img
alt="Open in Web" width="114" height="28"
src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a>&nbsp;<a
href="https://cursor.com/background-agent?bcId=bc-8f58544c-4504-5696-b018-da550b0b8144"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img
alt="Open in Cursor" width="131" height="28"
src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a>&nbsp;</div>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Jonas <JonasBa@users.noreply.github.com>
`fromtimestamp()` converts a POSIX timestamp to local time; using
`date.replace(tzinfo=timezone.utc)` does ensure it is UTC, but unless
local time is UTC, it will result in an incorrect timestamp.
We're usually running UTC, so not a practical concern, but it's easy to
be correct here regardless of environment.
@pull pull bot locked and limited conversation to collaborators Mar 20, 2026
@pull pull bot added the ⤵️ pull label Mar 20, 2026
@pull pull bot merged commit 39d4ea9 into KingDEV95:master Mar 20, 2026
@github-actions github-actions bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Mar 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⤵️ pull Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants