[web] Fix reacting to robotext
ClosedPublic
Actions

Authored by ashoat on Mon, Oct 14, 11:44 AM.

Tags

None

Referenced Files

	F2982456: D13708.id45177.diff
	Wed, Oct 16, 3:34 AM

	F2982148: D13708.diff
	Wed, Oct 16, 1:47 AM

	Unknown Object (File)
	Tue, Oct 15, 3:53 AM

	Unknown Object (File)
	Tue, Oct 15, 3:38 AM

	Unknown Object (File)
	Tue, Oct 15, 3:18 AM

	Unknown Object (File)
	Tue, Oct 15, 3:15 AM

	Unknown Object (File)
	Tue, Oct 15, 2:48 AM

	Unknown Object (File)
	Tue, Oct 15, 2:47 AM

View All 9 Files

Subscribers

None

Details

Reviewers

tomek
angelika
kamil

Commits

rCOMMfd480e62c125: [web] Fix reacting to robotext

Summary

Similar to the parent diff, this was broken by my work on combining robotext messages.

Depends on D13707

Test Plan

Confirm that the user can react to non-combined robotext on web again

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Mon, Oct 14, 11:44 AM

ashoat added a child revision: D13709: [native] Fix reacting to robotext when sidebar is disabled.Mon, Oct 14, 11:47 AM

Harbormaster completed remote builds in B32239: Diff 45177.Mon, Oct 14, 12:01 PM

ashoat requested review of this revision.Mon, Oct 14, 12:01 PM

kamil accepted this revision.Tue, Oct 15, 1:10 AM

This revision is now accepted and ready to land.Tue, Oct 15, 1:10 AM

Closed by commit rCOMMfd480e62c125: [web] Fix reacting to robotext (authored by ashoat). · Explain WhyTue, Oct 15, 5:46 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMfd480e62c125: [web] Fix reacting to robotext.

Revision Contents
Changeset List

Path

Size

web/

tooltips/

tooltip-action-utils.js

2 lines

Diff 45197

View Options

web/tooltips/tooltip-action-utils.js

[web] Fix reacting to robotextClosedPublicActions