Skip to content

Commit 953b6f6

Browse files
authored
fix(timetamped-comments): replacing dash container class with media container (#4409)
fix(timetamped-comments): using media container instead of dash container
1 parent 7f16b5a commit 953b6f6

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

src/components/form-elements/draft-js-mention-selector/DraftJSMentionSelector.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -463,9 +463,9 @@ class DraftJSMentionSelector extends React.Component<Props, State> {
463463
};
464464

465465
getVideoTimestamp = (): videoTimestamp => {
466-
const mediaDashContainer: ?HTMLElement = document.querySelector('.bp-media-dash');
466+
const videoContainer: ?HTMLElement = document.querySelector('.bp-media-container');
467467
// $FlowFixMe
468-
const video: ?HTMLVideoElement = mediaDashContainer?.querySelector('video');
468+
const video: ?HTMLVideoElement = videoContainer?.querySelector('video');
469469

470470
const currentTime = video?.currentTime || 0;
471471

src/elements/content-sidebar/activity-feed/comment-form/stories/CommentForm.stories.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ export const VideoFile = () => {
6666

6767
return (
6868
<div>
69-
<div className="bp-media-dash" style={{ width: '100px', height: '100px', marginBottom: '10px' }}>
69+
<div className="bp-media-container" style={{ width: '100px', height: '100px', marginBottom: '10px' }}>
7070
{/* eslint-disable-next-line jsx-a11y/media-has-caption */}
7171
<video
7272
src="//cdn03.boxcdn.net/sites/default/files/homepage/v2/images/hero/run/laptop-screen-1680-v2@1x.mp4"

src/elements/content-sidebar/activity-feed/utils/__tests__/formatTaggedMessage.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ describe('elements/content-sidebar/ActivityFeed/utils/formatTaggedMessage', () =
9898
const mockMediaDashContainer = { querySelector: () => mockVideo };
9999

100100
jest.spyOn(document, 'querySelector').mockImplementation(query => {
101-
if (query === '.bp-media-dash') {
101+
if (query === '.bp-media-container') {
102102
return mockMediaDashContainer;
103103
}
104104

src/elements/content-sidebar/activity-feed/utils/formatTaggedMessage.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ const formatTimestamp = (text: string, timestamp: string, intl: IntlShape): Reac
4040

4141
const handleClick = (e: SyntheticMouseEvent<HTMLButtonElement>): void => {
4242
e.preventDefault();
43-
const videoContainer = document.querySelector('.bp-media-dash');
43+
const videoContainer = document.querySelector('.bp-media-container');
4444
if (videoContainer) {
4545
// $FlowFixMe: querySelector('video') returns an HTMLVideoElement
4646
const video: ?HTMLVideoElement = videoContainer.querySelector('video');

0 commit comments

Comments
 (0)