Skip to content

Conversation

@nsdeschenes
Copy link
Contributor

This PR adds in some refinements to the cross event dropdown, such as proper labels, ability to add a second cross event, and disabling itself when there are two cross events present.

I also moved the search section of the spans tab into its own file to keep things a bit more tidy, as well as refactoring the spans tab tests to use the additionalWrapper arg, over wrapping in the wrapper saving a couple lines.

Ticket: EXP-618

@linear
Copy link

linear bot commented Nov 27, 2025

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 27, 2025
@nsdeschenes
Copy link
Contributor Author

@sentry review

@nsdeschenes nsdeschenes marked this pull request as ready for review November 27, 2025 17:11
@nsdeschenes nsdeschenes requested a review from a team as a code owner November 27, 2025 17:11
@nsdeschenes nsdeschenes force-pushed the nd/exp-618/feat-add-cross-event-dropdown-functionality branch from f2a401a to 2dcbb84 Compare November 28, 2025 15:37
@nsdeschenes nsdeschenes force-pushed the nd/exp-618/feat-add-cross-event-dropdown-functionality branch from 2dcbb84 to 6bd0ad2 Compare December 1, 2025 12:38
@nsdeschenes nsdeschenes merged commit 7d243b2 into master Dec 1, 2025
48 checks passed
@nsdeschenes nsdeschenes deleted the nd/exp-618/feat-add-cross-event-dropdown-functionality branch December 1, 2025 15:36
jerryzhou196 pushed a commit that referenced this pull request Dec 1, 2025
This PR adds in some refinements to the cross event dropdown, such as
proper labels, ability to add a second cross event, and disabling itself
when there are two cross events present.

I also moved the search section of the spans tab into its own file to
keep things a bit more tidy, as well as refactoring the spans tab tests
to use the `additionalWrapper` arg, over wrapping in the wrapper saving
a couple lines.

Ticket: EXP-618
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants