Page MenuHomePhabricator
Feed Advanced Search

Mar 26 2023

ginsu updated the diff for D7137: [native] render thread avatars for parent/child threads in thread settings.

address feedback

Mar 26 2023, 12:13 AM
ginsu attached a referenced file: F445626: Screenshot 2023-03-24 at 3.04.22 PM.png.
Mar 26 2023, 12:12 AM
ginsu attached a referenced file: F445627: Screenshot 2023-03-26 at 3.11.51 AM.png.
Mar 26 2023, 12:12 AM
ginsu updated the test plan for D7136: [native] render thread avatars in community drawer.
Mar 26 2023, 12:12 AM
ginsu added a reviewer for D7136: [native] render thread avatars in community drawer: inka.
Mar 26 2023, 12:09 AM
ginsu updated the diff for D7136: [native] render thread avatars in community drawer.

address feedback

Mar 26 2023, 12:09 AM
ginsu updated the diff for D7135: [native] render thread avatar in CHANNEL AVATAR thread settings section.

address feedback

Mar 26 2023, 12:05 AM
ginsu attached a referenced file: F445550: Screenshot 2023-03-25 at 11.08.46 PM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445551: Screenshot 2023-03-25 at 10.39.40 PM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445552: Screenshot 2023-03-25 at 11.09.09 PM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445554: Screenshot 2023-03-25 at 10.55.13 PM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445555: Screenshot 2023-03-25 at 11.10.05 PM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445556: Screenshot 2023-03-25 at 10.53.19 PM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445558: Screenshot 2023-03-26 at 2.56.38 AM.png.
Mar 26 2023, 12:02 AM
ginsu attached a referenced file: F445562: Screenshot 2023-03-26 at 3.01.08 AM.png.
Mar 26 2023, 12:02 AM
ginsu retitled D7134: [native] update message list header to new design from [native] render thread avatar in message list header title to [native] update message list header to new design.
Mar 26 2023, 12:02 AM

Mar 25 2023

ginsu updated the diff for D7134: [native] update message list header to new design.

update message list header to new desgin

Mar 25 2023, 11:44 PM
ginsu abandoned D7133: [lib] introduce getAvatarForThreadEntity function.

No longer necessary with D7178

Mar 25 2023, 11:41 PM
ginsu added a comment to D7132: [lib] introduce useGetAvatarForThread hook.

Mostly questions

Mar 25 2023, 11:41 PM
ginsu updated the diff for D7132: [lib] introduce useGetAvatarForThread hook.

remove stray ?

Mar 25 2023, 11:37 PM
ginsu updated the summary of D7178: [lib] introduce getUserAvatarForThread function.
Mar 25 2023, 11:35 PM
ginsu attached a referenced file: F445490: Screenshot 2023-03-24 at 3.04.22 PM.png.
Mar 25 2023, 11:34 PM
ginsu attached a referenced file: F445489: Screenshot 2023-03-24 at 3.04.13 PM.png.
Mar 25 2023, 11:34 PM
ginsu updated the summary of D7132: [lib] introduce useGetAvatarForThread hook.
Mar 25 2023, 11:34 PM
ginsu updated the diff for D7132: [lib] introduce useGetAvatarForThread hook.

change order

Mar 25 2023, 11:24 PM
ginsu updated the diff for D7132: [lib] introduce useGetAvatarForThread hook.

new version of useGetAvatarForThread

Mar 25 2023, 11:20 PM
ginsu added inline comments to D7178: [lib] introduce getUserAvatarForThread function.
Mar 25 2023, 11:17 PM
ginsu attached a referenced file: F445426: Screenshot 2023-03-26 at 2.03.53 AM.png.
Mar 25 2023, 11:16 PM
ginsu attached a referenced file: F445425: Screenshot 2023-03-26 at 2.03.55 AM.png.
Mar 25 2023, 11:16 PM
ginsu attached a referenced file: F445424: Screenshot 2023-03-26 at 2.05.59 AM.png.
Mar 25 2023, 11:16 PM
ginsu attached a referenced file: F445423: Screenshot 2023-03-26 at 2.05.44 AM.png.
Mar 25 2023, 11:16 PM
ginsu updated the summary of D7178: [lib] introduce getUserAvatarForThread function.
Mar 25 2023, 11:16 PM
ginsu requested review of D7178: [lib] introduce getUserAvatarForThread function.
Mar 25 2023, 11:04 PM
ginsu requested review of D7177: [keyserver/lib/native/web] introduce color-utils and move color functions from thread-utils to color-utils.
Mar 25 2023, 10:59 PM

Mar 23 2023

ginsu closed D7067: [native] render user avatars in chat screen.
Mar 23 2023, 11:51 PM
ginsu committed rCOMM6f269a0c7bd9: [native] render user avatars in chat screen (authored by ginsu).
[native] render user avatars in chat screen
Mar 23 2023, 11:51 PM
ginsu added a comment to D7067: [native] render user avatars in chat screen.

One last thing that would be great if you could test: what happens if you upload a single very "thin" photo? I want to make sure it doesn't expand to take the whole width and then becomes super tall. (Or at least, that there's no regression in this.)

Sorry for all the specific test plan requests, I just remember there being a lot of edge cases with this code from back when I was working on it.

Mar 23 2023, 11:49 PM
ginsu attached a referenced file: F442621: Screen Recording 2023-03-24 at 2.37.39 AM.mov.
Mar 23 2023, 11:47 PM
ginsu attached a referenced file: F442620: Screen Recording 2023-03-24 at 2.40.23 AM.mov.
Mar 23 2023, 11:47 PM
ginsu updated the test plan for D7067: [native] render user avatars in chat screen.
Mar 23 2023, 11:47 PM
ginsu added inline comments to D7067: [native] render user avatars in chat screen.
Mar 23 2023, 11:44 PM
ginsu closed D7160: Revert "[web] render avatar in account screen".
Mar 23 2023, 1:25 PM
ginsu added a reverting change for rCOMM8588624614a4: [web] render avatar in account screen: rCOMM3a1e14753e51: Revert "[web] render avatar in account screen".
Mar 23 2023, 1:25 PM
ginsu committed rCOMM3a1e14753e51: Revert "[web] render avatar in account screen" (authored by ginsu).
Revert "[web] render avatar in account screen"
Mar 23 2023, 1:25 PM
ginsu closed D7159: Revert "[web] render user avatars in chat screen".
Mar 23 2023, 1:25 PM
ginsu added a reverting change for rCOMM5db4c70262ee: [web] render user avatars in chat screen: rCOMM7a811e67f0ee: Revert "[web] render user avatars in chat screen".
Mar 23 2023, 1:25 PM
ginsu committed rCOMM7a811e67f0ee: Revert "[web] render user avatars in chat screen" (authored by ginsu).
Revert "[web] render user avatars in chat screen"
Mar 23 2023, 1:25 PM
ginsu closed D7158: Revert "[keyserver/lib/web/native] fetch user avatar column from users table for remaning user fetchers".
Mar 23 2023, 1:25 PM
ginsu added a reverting change for rCOMM126db3bda66c: [keyserver/lib/web/native] fetch user avatar column from users table for…: rCOMMc3011806b124: Revert "[keyserver/lib/web/native] fetch user avatar column from users table….
Mar 23 2023, 1:25 PM
ginsu committed rCOMMc3011806b124: Revert "[keyserver/lib/web/native] fetch user avatar column from users table… (authored by ginsu).
Revert "[keyserver/lib/web/native] fetch user avatar column from users table…
Mar 23 2023, 1:25 PM
ginsu added a reverting change for rCOMM8588624614a4: [web] render avatar in account screen: D7160: Revert "[web] render avatar in account screen".
Mar 23 2023, 1:19 PM
ginsu added a reverting change for rCOMM5db4c70262ee: [web] render user avatars in chat screen: D7159: Revert "[web] render user avatars in chat screen".
Mar 23 2023, 1:18 PM
ginsu added a reverting change for rCOMM126db3bda66c: [keyserver/lib/web/native] fetch user avatar column from users table for…: D7158: Revert "[keyserver/lib/web/native] fetch user avatar column from users table for remaning user fetchers".
Mar 23 2023, 1:17 PM
ginsu committed rCOMM126db3bda66c: [keyserver/lib/web/native] fetch user avatar column from users table for… (authored by ginsu).
[keyserver/lib/web/native] fetch user avatar column from users table for…
Mar 23 2023, 1:14 PM
ginsu committed rCOMM5db4c70262ee: [web] render user avatars in chat screen (authored by ginsu).
[web] render user avatars in chat screen
Mar 23 2023, 1:14 PM
ginsu committed rCOMM8588624614a4: [web] render avatar in account screen (authored by ginsu).
[web] render avatar in account screen
Mar 23 2023, 1:14 PM
ginsu accepted D7152: [keyserver] Introduce `updateUserAvatar` updater and use in `updateUserAvatarResponder`.
Mar 23 2023, 11:58 AM
ginsu updated the diff for D7067: [native] render user avatars in chat screen.

memoize MessageTooltipButtonAvatar

Mar 23 2023, 9:59 AM
ginsu added inline comments to D7067: [native] render user avatars in chat screen.
Mar 23 2023, 9:51 AM
ginsu attached a referenced file: F441961: Screenshot 2023-03-23 at 12.44.09 PM.png.
Mar 23 2023, 9:45 AM
ginsu attached a referenced file: F441960: Screenshot 2023-03-23 at 12.44.26 PM.png.
Mar 23 2023, 9:45 AM
ginsu attached a referenced file: F441956: Screen Recording 2023-03-23 at 12.42.26 PM.mov.
Mar 23 2023, 9:45 AM
ginsu attached a referenced file: F441951: Screen Recording 2023-03-23 at 12.35.42 PM.mov.
Mar 23 2023, 9:45 AM
ginsu attached a referenced file: F441950: Screen Recording 2023-03-23 at 12.35.10 PM.mov.
Mar 23 2023, 9:45 AM
ginsu attached a referenced file: F441949: Screen Recording 2023-03-23 at 12.34.55 PM.mov.
Mar 23 2023, 9:45 AM
ginsu updated the test plan for D7067: [native] render user avatars in chat screen.
Mar 23 2023, 9:45 AM
ginsu updated the diff for D7067: [native] render user avatars in chat screen.

address comments

Mar 23 2023, 9:37 AM
ginsu added inline comments to D7067: [native] render user avatars in chat screen.
Mar 23 2023, 9:12 AM
ginsu attached a referenced file: F441840: Screenshot 2023-03-23 at 11.35.18 AM.png.
Mar 23 2023, 8:39 AM
ginsu added inline comments to D7134: [native] update message list header to new design.
Mar 23 2023, 8:39 AM
ginsu added a comment to D7133: [lib] introduce getAvatarForThreadEntity function.

you almost certainly need to be looking at threadInfoFromRawThreadInfo.

Mar 23 2023, 8:39 AM
ginsu attached a referenced file: F441194: Screenshot 2023-03-22 at 7.44.33 PM.png.
Mar 23 2023, 8:38 AM
ginsu added inline comments to D7067: [native] render user avatars in chat screen.
Mar 23 2023, 8:38 AM

Mar 22 2023

ginsu attached a referenced file: F441060: Screen Recording 2023-03-22 at 6.28.52 PM.mov.
Mar 22 2023, 3:43 PM
ginsu attached a referenced file: F441058: Screenshot 2023-03-21 at 5.31.04 PM.png.
Mar 22 2023, 3:43 PM
ginsu added a comment to D7067: [native] render user avatars in chat screen.

My last proposed solution to fix the chat bubble poking out of the message box is to make the message box a flex container and align the inner text messages (either flex-end or flex-start) depending on if the message came from the viewer or not. This solution is very similar to how we position the InlineEngagement component, which inspired me to implement the solution this way.

Mar 22 2023, 3:43 PM
ginsu updated the diff for D7067: [native] render user avatars in chat screen.

add feature flags and fix poking chat bubble

Mar 22 2023, 3:42 PM
ginsu closed D7131: [lib] introduce getDefaultAvatar helper function.
Mar 22 2023, 1:23 PM
ginsu committed rCOMM9ea6a06baee7: [lib] introduce getDefaultAvatar helper function (authored by ginsu).
[lib] introduce getDefaultAvatar helper function
Mar 22 2023, 1:23 PM
ginsu attached a referenced file: F439935: Screenshot 2023-03-22 at 4.01.00 AM.png.
Mar 22 2023, 1:03 AM
ginsu attached a referenced file: F439937: Screenshot 2023-03-22 at 4.02.25 AM.png.
Mar 22 2023, 1:03 AM
ginsu attached a referenced file: F439936: Screenshot 2023-03-22 at 4.01.48 AM.png.
Mar 22 2023, 1:03 AM
ginsu attached a referenced file: F439938: Screenshot 2023-03-22 at 4.02.27 AM.png.
Mar 22 2023, 1:03 AM
ginsu updated the summary of D7138: [native] render the rest of the thread avatars.
Mar 22 2023, 1:03 AM
ginsu attached a referenced file: F439931: Screenshot 2023-03-22 at 3.56.31 AM.png.
Mar 22 2023, 12:57 AM
ginsu attached a referenced file: F439930: Screenshot 2023-03-22 at 3.55.48 AM.png.
Mar 22 2023, 12:57 AM
ginsu updated the summary of D7137: [native] render thread avatars for parent/child threads in thread settings.
Mar 22 2023, 12:57 AM
ginsu attached a referenced file: F439925: Screenshot 2023-03-22 at 3.48.46 AM.png.
Mar 22 2023, 12:54 AM
ginsu attached a referenced file: F439926: Screenshot 2023-03-22 at 3.48.30 AM.png.
Mar 22 2023, 12:54 AM
ginsu updated the summary of D7136: [native] render thread avatars in community drawer.
Mar 22 2023, 12:54 AM
ginsu updated the diff for D7135: [native] render thread avatar in CHANNEL AVATAR thread settings section.

update prop types for thread settings avatar

Mar 22 2023, 12:45 AM
ginsu attached a referenced file: F439888: Screenshot 2023-03-22 at 3.34.54 AM.png.
Mar 22 2023, 12:35 AM
ginsu attached a referenced file: F439887: Screenshot 2023-03-22 at 3.34.29 AM.png.
Mar 22 2023, 12:35 AM
ginsu updated the summary of D7135: [native] render thread avatar in CHANNEL AVATAR thread settings section.
Mar 22 2023, 12:35 AM
ginsu attached a referenced file: F439873: Screenshot 2023-03-22 at 3.25.12 AM.png.
Mar 22 2023, 12:27 AM
ginsu attached a referenced file: F439874: Screenshot 2023-03-22 at 3.25.31 AM.png.
Mar 22 2023, 12:27 AM
ginsu attached a referenced file: F439876: Screenshot 2023-03-22 at 3.26.39 AM.png.
Mar 22 2023, 12:27 AM
ginsu attached a referenced file: F439871: Screenshot 2023-03-22 at 2.10.30 AM.png.
Mar 22 2023, 12:27 AM
ginsu attached a referenced file: F439872: Screenshot 2023-03-22 at 2.11.01 AM.png.
Mar 22 2023, 12:27 AM
ginsu updated the summary of D7134: [native] update message list header to new design.
Mar 22 2023, 12:26 AM