Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ref(replay): Refactor video-replay related css #82514

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ryan953
Copy link
Member

@ryan953 ryan953 commented Dec 23, 2024

This moves two bits of mobile-replay video-replay CSS closer to where it's needed.

first, i moved the .video-replayer class down from videoReplayerWithInteractions.tsx and into videoReplayer.tsx with the new name .video-replayer-wrapper.

Then i updated the selectors to use sibling selections. So if the video wrapper exists it'll kick in.

We want the opacity rule to kick in no matter the use-case, so that stays inside styles.tsx. But the absolute positioning is only for the older static/app/components/replays/replayPlayer.tsx and not the newer static/app/components/replays/player/replayPlayer.tsx.

The newer one, when it gets support for video replays, will use <Stacked> instead of postition:absolute; like this.

@ryan953 ryan953 requested a review from a team as a code owner December 23, 2024 03:06
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 23, 2024
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.

1 participant