Page MenuHomePhabricator
Differential D4923

[web] Remove old tooltip components
ClosedPublic
Actions

Authored by • jacek on Aug 23 2022, 5:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 11, 1:52 AM
Unknown Object (File)
Mon, Oct 28, 5:07 AM
Unknown Object (File)
Mon, Oct 28, 5:07 AM
Unknown Object (File)
Mon, Oct 28, 5:07 AM
Unknown Object (File)
Mon, Oct 28, 5:05 AM
Unknown Object (File)
Oct 25 2024, 4:49 PM
Unknown Object (File)
Oct 25 2024, 4:57 AM
Unknown Object (File)
Oct 23 2024, 9:58 AM
View All Files
Subscribers
ashoat

Details

Reviewers
tomek
atul
• abosh
Commits
rCOMMdbdac9d30be7: [web] Remove old tooltip components
Summary

Remove no longer used code related to tooltip.
At this point, only the new tooltip should be displayed.

The new component on the video:
https://linear.app/comm/issue/ENG-1575#comment-d3fe1db0

Test Plan

Tested the new tooltip on text, media and robotext messages with different sizes a location. Tested if all the actions work as before and the action list remains unchanged. The new tooltip should not cover inline sidebar if it exists.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• jacek created this revision.Aug 23 2022, 5:17 AM
• jacek held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 23 2022, 5:17 AM
Harbormaster completed remote builds in B11541: Diff 15864.Aug 23 2022, 5:26 AM
• jacek added a parent revision: D4922: [web] Remove timestamp tooltip from `ChatMessageList`.Aug 23 2022, 5:29 AM
• jacek added a child revision: D4924: [web] Redesign `InlineSidebar` with reactions support.
• jacek edited the summary of this revision. (Show Details)Aug 23 2022, 5:32 AM
• jacek published this revision for review.Aug 23 2022, 8:06 AM
tomek accepted this revision.Aug 26 2022, 9:36 AM

Tested if all the actions work as before.

Have you tested if the set of actions is exactly the same as before?

This revision is now accepted and ready to land.Aug 26 2022, 9:36 AM
• jacek edited the test plan for this revision. (Show Details)Sep 1 2022, 6:21 AM
• jacek edited the test plan for this revision. (Show Details)
In D4923#143832, @tomek wrote:

Tested if all the actions work as before.

Have you tested if the set of actions is exactly the same as before?

Yes, I've checked it

• jacek updated this revision to Diff 16202.Sep 1 2022, 7:16 AM

rebase

Harbormaster completed remote builds in B11799: Diff 16202.Sep 1 2022, 7:26 AM
Closed by commit rCOMMdbdac9d30be7: [web] Remove old tooltip components (authored by • jacek). · Explain WhySep 5 2022, 3:13 AM
This revision was automatically updated to reflect the committed changes.
• jacek added a commit: rCOMMdbdac9d30be7: [web] Remove old tooltip components.

Revision Contents
Changeset List

Diff 16299

View Options

web/chat/message-reply-button.react.js

Loading...
View Options

web/chat/message-timestamp-tooltip.css

Loading...
View Options

web/chat/message-timestamp-tooltip.react.js

Loading...
View Options

web/chat/tooltip-utils.js

Loading...
View Options

web/chat/tooltip.css

Loading...
View Options

web/chat/tooltip.react.js

Loading...