Skip to content

Conversation

@jerryzhou196
Copy link
Member

@jerryzhou196 jerryzhou196 commented Nov 28, 2025

REPLAY-814 - new replay UI demo

IGNORE THIS PR - this is purely to be deployed as a preview on branch

- Removed unnecessary alignment property from KeyMetrics.
- Introduced constants for navigation height and border styles in secondary navigation.
- Refactored ReplayDetails header to use Flex layout for improved structure and styling consistency.
- Updated header styles to utilize new constants for height and border, enhancing maintainability.
- Introduced conditional rendering for the header based on organization features.
- Replaced the existing header with a new styled component for improved layout.
- Updated styling for the new header to align with design specifications.
- Consolidated common styles for platform icons and counters.
- Removed InnerWrapper and Counter components to streamline the layout.
- Updated styling to enhance hover effects and maintain consistent sizing for platform icons.
…play details

- Removed unnecessary common styles and consolidated styling for platform icons and counters.
- Updated font sizes and weights in replay metadata for better readability.
- Adjusted placeholder dimensions in replay viewers and details user badge for consistency.
- Enhanced layout in replay details metadata and breadcrumbs for improved visual hierarchy.
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 28, 2025
@codecov
Copy link

codecov bot commented Nov 28, 2025

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
12739 2 12737 10
View the top 2 failed test(s) by shortest run time
replayDetailsUserBadge should hide LIVE badge when last received segment is more than 5 minutes ago
Stack Traces | 0.027s run time
Error: expect(received).toBeVisible()

received value must be an HTMLElement or an SVGElement.
Received has value: null
    at __EXTERNAL_MATCHER_TRAP__ (.../sentry/node_modules/.pnpm/expect@30.0..../expect/build/index.js:2237:22)
    at Object.throwingMatcher [as toBeVisible] (.../sentry/node_modules/.pnpm/expect@30.0..../expect/build/index.js:2238:6)
    at Object.toBeVisible (.../detail/header/replayDetailsUserBadge.spec.tsx:305:48)
replayDetailsUserBadge should show LIVE badge when last received segment is within 5 minutes
Stack Traces | 0.055s run time
Error: expect(received).toBeVisible()

received value must be an HTMLElement or an SVGElement.
Received has value: null
    at __EXTERNAL_MATCHER_TRAP__ (.../sentry/node_modules/.pnpm/expect@30.0..../expect/build/index.js:2237:22)
    at Object.throwingMatcher [as toBeVisible] (.../sentry/node_modules/.pnpm/expect@30.0..../expect/build/index.js:2238:6)
    at Object.toBeVisible (.../detail/header/replayDetailsUserBadge.spec.tsx:238:48)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

…d layout

- Updated padding values in the StyledFlex component to enhance visual consistency and alignment with design specifications.
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.

2 participants