Paths

Table of Contentst

-
web/
-
app.react.js

Differential D4919

[web] Wrap app content with `TooltipProvider`
ClosedPublic
Actions

Authored by • jacek on Aug 23 2022, 5:05 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Mar 16, 4:58 AM

	Unknown Object (File)
	Wed, Mar 12, 2:27 PM

	Unknown Object (File)
	Wed, Mar 12, 4:04 AM

	Unknown Object (File)
	Wed, Mar 5, 1:21 AM

	Unknown Object (File)
	Wed, Mar 5, 1:14 AM

	Unknown Object (File)
	Feb 19 2025, 8:14 PM

	Unknown Object (File)
	Feb 19 2025, 8:13 PM

	Unknown Object (File)
	Jan 31 2025, 5:51 PM

Subscribers

Details

Reviewers

tomek
atul
• abosh

Commits

rCOMMc12778cdcedd: [web] Wrap app content with `TooltipProvider`

Summary

Use TooltipProvider in main app component.

Test Plan

Flow; Tested with the rest of diffs stack.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jacek created this revision.Aug 23 2022, 5:05 AM

• jacek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptAug 23 2022, 5:05 AM

Harbormaster completed remote builds in B11537: Diff 15860.Aug 23 2022, 5:13 AM

• jacek added a parent revision: D4918: [web] Introduce `useMessageTooltipActions` hooks.Aug 23 2022, 5:28 AM

• jacek added a child revision: D4920: [web] Use new tooltip in composed message.

• jacek published this revision for review.Aug 23 2022, 8:06 AM

atul accepted this revision.Aug 25 2022, 10:21 PM

This revision is now accepted and ready to land.Aug 25 2022, 10:21 PM

tomek accepted this revision.Aug 26 2022, 8:56 AM

rebase

Harbormaster completed remote builds in B11795: Diff 16198.Sep 1 2022, 7:19 AM

Closed by commit rCOMMc12778cdcedd: [web] Wrap app content with `TooltipProvider` (authored by • jacek). · Explain WhySep 5 2022, 3:13 AM

This revision was automatically updated to reflect the committed changes.

• jacek added a commit: rCOMMc12778cdcedd: [web] Wrap app content with `TooltipProvider`.

Revision Contents
Changeset List

Path

Size

web/

17 lines

Diff 16295

web/app.react.js

This file was added.

				// 400KiB limit (in docs there is KB but they mean KiB) -
				// https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/ServiceQuotas.html
				// This includes both attribute names' and values' lengths
				pub const DDB_ITEM_SIZE_LIMIT: usize = 1024 * 400;

				// 4MB limit
				// WARNING: use keeping in mind that grpc adds its own headers to messages
				// https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md
				// so the message that actually is being sent over the network looks like this
				// [Compressed-Flag] [Message-Length] [Message]
				// [Compressed-Flag] 1 byte - added by grpc
				// [Message-Length] 4 bytes - added by grpc
				// [Message] N bytes - actual data
				// so for every message we get 5 additional bytes of data
				// as mentioned here
				// https://github.com/grpc/grpc/issues/15734#issuecomment-396962671
				// grpc stream may contain more than one message
				pub const GRPC_CHUNK_SIZE_LIMIT: usize = 4 * 1024 * 1024;
				pub const GRPC_METADATA_SIZE_PER_MESSAGE: usize = 5;