Page MenuHomePhabricator
Feed Advanced Search

Mar 15 2023

ginsu attached a referenced file: F428875: 336223399_740379237747150_8312429542900821292_n.jpg.
Mar 15 2023, 5:16 PM
ginsu attached a referenced file: F428877: 336353513_590716002715617_5037247747435841645_n.jpg.
Mar 15 2023, 5:16 PM
ginsu attached a referenced file: F428879: 336162309_908146963557437_5514534208953352892_n.jpg.
Mar 15 2023, 5:16 PM
ginsu added a comment to D7067: [native] render user avatars in chat screen.

I feel like we should still reduce the width for viewer-composed multimedia messages so all the multimedia messages have a consistently smaller size,

Mar 15 2023, 5:15 PM
ginsu attached a referenced file: F428871: Screenshot 2023-03-15 at 7.38.28 PM.png.
Mar 15 2023, 5:01 PM
ginsu planned changes to D7067: [native] render user avatars in chat screen.
Mar 15 2023, 5:01 PM
ginsu added inline comments to D7067: [native] render user avatars in chat screen.
Mar 15 2023, 2:27 PM
ginsu updated the summary of D7067: [native] render user avatars in chat screen.
Mar 15 2023, 2:26 PM
ginsu attached a referenced file: F428737: Screenshot 2023-03-15 at 4.06.49 PM.png.
Mar 15 2023, 2:26 PM
ginsu attached a referenced file: F428736: Screenshot 2023-03-15 at 2.13.21 PM.png.
Mar 15 2023, 2:26 PM
ginsu updated the summary of D7067: [native] render user avatars in chat screen.
Mar 15 2023, 2:26 PM
ginsu updated the diff for D7067: [native] render user avatars in chat screen.

fix maxWidth for chat bubble so it's not poking out anymore (see test plan for visulization)

Mar 15 2023, 2:17 PM
ginsu added a comment to D7067: [native] render user avatars in chat screen.

With the red border, is that a parent component of the message box? (Generally I would expect the parent component to enclose the child component.)

Mar 15 2023, 12:11 PM
ginsu attached a referenced file: F428518: Screenshot 2023-03-15 at 2.13.21 PM.png.
Mar 15 2023, 11:36 AM
ginsu planned changes to D7067: [native] render user avatars in chat screen.

Have you made sure that that the height measurement code is still working correctly?

Mar 15 2023, 11:35 AM
ginsu accepted D7074: [lib][keyserver] Update reaction permission checks to check REACT_TO_MESSAGE.

thanks for doing this!

Mar 15 2023, 9:19 AM
ginsu accepted D7073: [lib][keyserver] Introduce new threadPermissions types for reactions.
Mar 15 2023, 9:18 AM

Mar 14 2023

ginsu requested review of D7069: [native] render the rest of the user avatars.
Mar 14 2023, 2:18 PM
ginsu requested review of D7067: [native] render user avatars in chat screen.
Mar 14 2023, 1:52 PM
ginsu added a comment to D7054: [keyserver/lib] fetch avatar column from users table in fetchCurrentUserInfo.

Okay sounds good i'll have an update out shortly with a cleaner syntax

Mar 14 2023, 12:30 PM
ginsu added inline comments to D7054: [keyserver/lib] fetch avatar column from users table in fetchCurrentUserInfo.
Mar 14 2023, 8:38 AM
ginsu attached a referenced file: F427092: Screenshot 2023-03-14 at 11.12.09 AM.png.
Mar 14 2023, 8:34 AM
ginsu attached a referenced file: F427091: Screenshot 2023-03-14 at 11.08.57 AM.png.
Mar 14 2023, 8:34 AM
ginsu updated the summary of D7054: [keyserver/lib] fetch avatar column from users table in fetchCurrentUserInfo.
Mar 14 2023, 8:33 AM
ginsu updated the diff for D7054: [keyserver/lib] fetch avatar column from users table in fetchCurrentUserInfo.

address feedback

Mar 14 2023, 8:31 AM
ginsu attached a referenced file: F427065: Screenshot 2023-03-14 at 11.06.36 AM.png.
Mar 14 2023, 8:25 AM
ginsu updated the summary of D7053: [lib] introduce getAvatarForUser in avatar-utils.
Mar 14 2023, 8:25 AM
ginsu updated the diff for D7053: [lib] introduce getAvatarForUser in avatar-utils.

address feedback

Mar 14 2023, 8:24 AM

Mar 13 2023

ginsu added inline comments to D7054: [keyserver/lib] fetch avatar column from users table in fetchCurrentUserInfo.
Mar 13 2023, 3:14 PM
ginsu attached a referenced file: F426471: Screenshot 2023-03-13 at 5.55.15 PM.png.
Mar 13 2023, 3:10 PM
ginsu attached a referenced file: F426470: Screenshot 2023-03-13 at 5.55.07 PM.png.
Mar 13 2023, 3:10 PM
ginsu attached a referenced file: F426473: Screenshot 2023-03-13 at 6.02.57 PM.png.
Mar 13 2023, 3:10 PM
ginsu added inline comments to D7053: [lib] introduce getAvatarForUser in avatar-utils.
Mar 13 2023, 3:10 PM
ginsu added inline comments to D7053: [lib] introduce getAvatarForUser in avatar-utils.
Mar 13 2023, 2:06 PM
ginsu updated the summary of D7057: [native] render avatar in profile screen.
Mar 13 2023, 1:25 AM

Mar 12 2023

ginsu updated the summary of D7057: [native] render avatar in profile screen.
Mar 12 2023, 11:52 PM
ginsu requested review of D7057: [native] render avatar in profile screen.
Mar 12 2023, 11:52 PM
ginsu requested review of D7055: [native] introduce Avatar component.
Mar 12 2023, 11:22 PM
ginsu requested review of D7054: [keyserver/lib] fetch avatar column from users table in fetchCurrentUserInfo.
Mar 12 2023, 11:08 PM
ginsu added inline comments to D7053: [lib] introduce getAvatarForUser in avatar-utils.
Mar 12 2023, 10:47 PM
ginsu attached a referenced file: F424838: Screenshot 2023-03-13 at 1.22.55 AM.png.
Mar 12 2023, 10:46 PM
ginsu attached a referenced file: F424840: Screenshot 2023-03-13 at 1.43.52 AM.png.
Mar 12 2023, 10:46 PM
ginsu requested review of D7053: [lib] introduce getAvatarForUser in avatar-utils.
Mar 12 2023, 10:46 PM

Mar 10 2023

ginsu accepted D7033: [keyserver] Give all members permissions to add users to a community.
Mar 10 2023, 1:38 PM

Mar 9 2023

ginsu accepted D6964: [native] Added new editing messages type to MessageSpecs in C++.

Not an expert in C++ but your code follows the pattern I used when writing the C++ message spec for reactions. Would be helpful to get a second pair of eyes here

Mar 9 2023, 10:25 AM
ginsu requested changes to D6963: [lib] Added new MessageSpec for editing messages.

I think we should consider if we also need shimUnsupportedMessageInfo and unshimMessageInfo. These functions will help us support older native clients that haven't been updated yet.

Mar 9 2023, 10:21 AM
ginsu accepted D6962: [lib][keyserver] Add editing message type to messageTypes.

Accepting, but if what I suggested doesn't take too long might be worthwhile to update

Mar 9 2023, 10:04 AM
ginsu accepted D6961: [lib] Create types for message editing.
Mar 9 2023, 9:56 AM

Mar 8 2023

ginsu accepted D7015: [native] Replace LayoutAnimation for driving fixed Tooltip animation.
Mar 8 2023, 8:44 PM
ginsu accepted D7014: [native] Add transformX animation to ReactionSelectionPopover.
Mar 8 2023, 8:41 PM
ginsu accepted D7013: [native] Add transformY animation to ReactionSelectionPopover.
Mar 8 2023, 8:37 PM
ginsu accepted D7012: [native] Animate opacity and scale of ReactionSelectionPopover.
Mar 8 2023, 8:34 PM
ginsu accepted D7011: [native] Return popoverLocation from useReactionSelectionPopoverPosition.
Mar 8 2023, 8:31 PM
ginsu accepted D7010: [native] Dedup useReactionSelectionPopoverPosition calls.
Mar 8 2023, 8:28 PM

Mar 2 2023

ginsu closed D6634: [web] implement emoji keyboard position into message tooltip.
Mar 2 2023, 12:34 PM
ginsu committed rCOMMe1ee9c65d196: [web] implement emoji keyboard position into message tooltip (authored by ginsu).
[web] implement emoji keyboard position into message tooltip
Mar 2 2023, 12:34 PM
ginsu closed D6632: [web] introduce the getEmojiKeyboardPosition function.
Mar 2 2023, 12:34 PM
ginsu committed rCOMM83c353c19ccc: [web] introduce the getEmojiKeyboardPosition function (authored by ginsu).
[web] introduce the getEmojiKeyboardPosition function
Mar 2 2023, 12:34 PM
ginsu updated the diff for D6634: [web] implement emoji keyboard position into message tooltip.

rebase before landing

Mar 2 2023, 12:32 PM
ginsu updated the diff for D6632: [web] introduce the getEmojiKeyboardPosition function.

rebase before landing

Mar 2 2023, 12:31 PM

Feb 28 2023

ginsu added inline comments to D6634: [web] implement emoji keyboard position into message tooltip.
Feb 28 2023, 10:15 PM
ginsu updated the diff for D6634: [web] implement emoji keyboard position into message tooltip.

address feedback

Feb 28 2023, 10:11 PM
ginsu added a comment to D6634: [web] implement emoji keyboard position into message tooltip.

Does D6632 need to be updated so that getEmojiKeyboardPosition no longer looks at emojiKeyboard.getBoundingClientRect()?

Feb 28 2023, 2:48 PM
ginsu abandoned D6905: [ignore] for atul to patch.
Feb 28 2023, 1:35 PM
ginsu attached a referenced file: F407335: Screenshot 2023-02-28 at 3.56.11 PM.png.
Feb 28 2023, 1:35 PM
ginsu attached a referenced file: F407245: Screenshot 2023-02-28 at 3.53.04 PM.png.
Feb 28 2023, 1:35 PM
ginsu attached a referenced file: F407233: Screenshot 2023-02-28 at 3.32.13 PM.png.
Feb 28 2023, 1:34 PM
ginsu added a comment to D6634: [web] implement emoji keyboard position into message tooltip.

I attempted to address the feedback above, but still running into one issue where the node passed through the ref still has a height and width of zero. (I saw this by logging node.getBoundingClientRect() in emojiKeyboardRef

Feb 28 2023, 1:34 PM
ginsu updated the diff for D6634: [web] implement emoji keyboard position into message tooltip.

address feedback

Feb 28 2023, 1:34 PM

Feb 27 2023

ginsu requested review of D6905: [ignore] for atul to patch.
Feb 27 2023, 3:18 PM

Feb 26 2023

ginsu closed D6830: [keyserver] introduce sendRescindNotifs.
Feb 26 2023, 1:06 PM
ginsu closed D6829: [keyserver] convert existing logic for building userPushInfoPromises to an async function called generateNotifUserInfoPromise.
Feb 26 2023, 1:06 PM
ginsu committed rCOMM9518dca6bbf6: [keyserver] introduce sendRescindNotifs (authored by ginsu).
[keyserver] introduce sendRescindNotifs
Feb 26 2023, 1:06 PM
ginsu committed rCOMM00510eaa214f: [keyserver] convert existing logic for building userPushInfoPromises to an… (authored by ginsu).
[keyserver] convert existing logic for building userPushInfoPromises to an…
Feb 26 2023, 1:06 PM
ginsu closed D6307: [lib] modify reaction message generatesNotifs function to return a pushType based on the reaction action.
Feb 26 2023, 1:06 PM
ginsu committed rCOMM1f10bbfc7149: [lib] modify reaction message generatesNotifs function to return a pushType… (authored by ginsu).
[lib] modify reaction message generatesNotifs function to return a pushType…
Feb 26 2023, 1:06 PM
ginsu closed D6875: [lib/native/web] switch reactions data structure from map to plain object.
Feb 26 2023, 1:00 PM
ginsu committed rCOMMbe977b7db9fb: [lib/native/web] switch reactions data structure from map to plain object (authored by ginsu).
[lib/native/web] switch reactions data structure from map to plain object
Feb 26 2023, 1:00 PM
ginsu updated the diff for D6830: [keyserver] introduce sendRescindNotifs.

rebase before landing

Feb 26 2023, 11:13 AM
ginsu updated the diff for D6829: [keyserver] convert existing logic for building userPushInfoPromises to an async function called generateNotifUserInfoPromise.

rebase before landing

Feb 26 2023, 11:12 AM
ginsu updated the diff for D6307: [lib] modify reaction message generatesNotifs function to return a pushType based on the reaction action.

rebase before landing

Feb 26 2023, 11:09 AM
ginsu updated the diff for D6875: [lib/native/web] switch reactions data structure from map to plain object.

rebase before landing

Feb 26 2023, 11:04 AM

Feb 24 2023

ginsu retitled D6829: [keyserver] convert existing logic for building userPushInfoPromises to an async function called generateNotifUserInfoPromise from [keyserver] convert existing logic for building userPushInfoPromises to an IIFE called generateNotifUserInfoPromise to [keyserver] convert existing logic for building userPushInfoPromises to an async function called generateNotifUserInfoPromise.
Feb 24 2023, 4:55 PM
ginsu updated the diff for D6829: [keyserver] convert existing logic for building userPushInfoPromises to an async function called generateNotifUserInfoPromise.

address ashoat's comments

Feb 24 2023, 4:54 PM
ginsu updated the test plan for D6875: [lib/native/web] switch reactions data structure from map to plain object.
Feb 24 2023, 4:49 PM
ginsu updated the diff for D6875: [lib/native/web] switch reactions data structure from map to plain object.

address ashoat's comments

Feb 24 2023, 4:48 PM
ginsu attached a referenced file: F401166: Screenshot 2023-02-24 at 7.41.15 PM.png.
Feb 24 2023, 4:45 PM
ginsu attached a referenced file: F401164: Screenshot 2023-02-24 at 7.40.50 PM.png.
Feb 24 2023, 4:45 PM
ginsu added inline comments to D6634: [web] implement emoji keyboard position into message tooltip.
Feb 24 2023, 4:45 PM

Feb 23 2023

ginsu requested review of D6875: [lib/native/web] switch reactions data structure from map to plain object.
Feb 23 2023, 2:58 PM
ginsu attached a referenced file: F398870: Screenshot 2023-02-23 at 11.57.29 AM.png.
Feb 23 2023, 8:58 AM
ginsu attached a referenced file: F398869: Screenshot 2023-02-23 at 11.57.17 AM.png.
Feb 23 2023, 8:58 AM
ginsu updated the test plan for D6830: [keyserver] introduce sendRescindNotifs.
Feb 23 2023, 8:58 AM
ginsu updated the diff for D6830: [keyserver] introduce sendRescindNotifs.

address ashoats comments

Feb 23 2023, 8:55 AM
ginsu added inline comments to D6830: [keyserver] introduce sendRescindNotifs.
Feb 23 2023, 8:54 AM
ginsu updated the diff for D6829: [keyserver] convert existing logic for building userPushInfoPromises to an async function called generateNotifUserInfoPromise.

address ashoat's comments

Feb 23 2023, 8:52 AM
ginsu closed D6838: [landing] remove przemek from landing page.
Feb 23 2023, 8:05 AM
ginsu committed rCOMM0886e8c04ba1: [landing] remove przemek from landing page (authored by ginsu).
[landing] remove przemek from landing page
Feb 23 2023, 8:05 AM

Feb 22 2023

ginsu requested review of D6838: [landing] remove przemek from landing page.
Feb 22 2023, 1:28 PM
ginsu closed D6786: [web] factor out app container position info logic out of tooltip utils.
Feb 22 2023, 12:55 PM