[web] Fix multimedia `onRemove` event propagation
ClosedPublic
Actions

Authored by • jacek on Jun 3 2022, 3:53 AM.

Tags

None

Referenced Files

	F2752240: D4209.diff
	Wed, Sep 18, 3:39 PM

	Unknown Object (File)
	Mon, Sep 2, 5:45 PM

	Unknown Object (File)
	Fri, Aug 30, 12:50 PM

	Unknown Object (File)
	Sat, Aug 24, 11:36 PM

	Unknown Object (File)
	Aug 11 2024, 7:21 PM

	Unknown Object (File)
	Aug 11 2024, 7:21 PM

	Unknown Object (File)
	Aug 11 2024, 7:21 PM

	Unknown Object (File)
	Aug 11 2024, 7:17 PM

Subscribers

Details

Reviewers

tomek
atul

Commits

rCOMM5e7526cca2b4: [web] Fix multimedia `onRemove` event propagation

Summary

Fix for multimedia remove issue: https://linear.app/comm/issue/ENG-1224/uploaded-media-are-displayed-in-fullscreen-modal-when-trying-to-remove

Test Plan

Try to upload and remove that multimedia on web. It should be removed without being opened.

Diff Detail

Repository

rCOMM Comm

Branch

jacek/media-fix

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• jacek created this revision.Jun 3 2022, 3:53 AM

Herald added subscribers: • abosh, • adrian, ashoat. · View Herald TranscriptJun 3 2022, 3:53 AM

Harbormaster completed remote builds in B9617: Diff 13329.Jun 3 2022, 4:00 AM

• jacek requested review of this revision.Jun 3 2022, 4:00 AM

tomek accepted this revision.Jun 3 2022, 4:45 AM

This revision is now accepted and ready to land.Jun 3 2022, 4:45 AM

Closed by commit rCOMM5e7526cca2b4: [web] Fix multimedia `onRemove` event propagation (authored by • jacek). · Explain WhyJun 7 2022, 2:50 AM

This revision was automatically updated to reflect the committed changes.

• jacek added a commit: rCOMM5e7526cca2b4: [web] Fix multimedia `onRemove` event propagation.

Revision Contents
Changeset List

Path

Size

web/

media/

multimedia.react.js

3 lines

Diff 13329

View Options

web/media/multimedia.react.js

[web] Fix multimedia `onRemove` event propagationClosedPublicActions