mozilla :: #activity-stream

11 Oct 2017
14:22ursulaMardak: i'm requesting uplift for the background page thumbs fix now, but i can't seem to reproduce the bug with or without the patch applied
14:22ursulaso i can't *really* verify it in nightly
15:09Mardakursula: can ni? reporter although you should be able to manually check by calling captureIfMissing over 100 times and looking at about:memory
15:22Mardakursula: oh it hasn't made it to nightly yet
15:23ursulaoh i guess mc doesn't count as nightly right
15:25ursulai was testing in mc tho, and still no luck
15:27streaminatorandreio: Hey, your patch was approved! https://github.com/mozilla/activity-stream/pull/3675
15:37Mardakursula: eh. good enough, request approval ;)
15:53ursulaMardak: oh you got it!! nice
15:53ursulai tried doing that too but didn't have any luck :(
16:31bsilverbergMeeting notes for newTabSection API: https://docs.google.com/a/mozilla.com/document/d/1PEYUdOk481P2ZEXi414fPp_wA0yx-0-J8Te3z_58SXI/edit?usp=sharing
17:46bsilverbergk88hudson: Although we're looking at landing an API to interact with Activity Stream, AS itself will remain a system add-on, correct?
17:52ursulabsilverberg: that's right
17:54bsilverbergThanks ursula
18:01dmosehttps://sql.telemetry.mozilla.org/dashboard/activity-stream-system-addon-performance?p_xaxis=addon_version
18:11k88hudsonhttps://public.etherpad-mozilla.org/p/newtabfaster
18:15k88hudsonhttps://drive.google.com/open?id=0B9Spnx3qAl7pdnMyMks3UHExV3M
18:36nanjdmose: sorry, i just dropped offline
18:36dmosenanj: heh, no worries
18:36dmoseit's not important
18:38nanjdmose: cool, let me know if you want to discuss about that patch
18:38dmoseok, will do
18:42Mardakdid everyone leave k88's room?
18:42ursulaMardak: yeah she had another meeting
18:42Mardakdesign says to show the placeholders
18:42Mardaki provided a video on slack if people want to see
18:42Mardaki opened up 10 tabs with 6 different configurations
18:42ursulaplaceholders and no fade?
18:42Mardak"no hide" is the last one
18:43Mardakursula: yeah. "no hide" = yes placeholders, no fade (last video segment). preferable over "no fade no delay" (2nd last video segment)
18:43ursularight
18:44Mardaka rough initial count shows 300ms for tiles to finish loading vs 300ms for placeholders to appear and another 300ms for content to show up
18:45ursulaand the placeholders come up in the pre-rendered version right?
18:46Mardakyes
19:02nanjdmose: tspurway: meeting time for exp-services
19:02Mardakk88hudson: here's removing the hiding that we'll need to backport to activity stream (and maybe remove the logic the body-wrapper stuff ? ) https://reviewboard.mozilla.org/r/188458/diff/1#index_header
19:05dmoseMardak: i have a couple of other ideas of things we can try too, i'll put them in here after my next week
19:05dmose(in addition, potentially)
19:10dmosehttps://public.etherpad-mozilla.org/p/newtabfaster#lineNumber=12
19:20k88hudsonMardak: sorry i'm still in meetings
19:20k88hudsonMardak: but yeah that's right
19:24k88hudsonMardak: do we want to remove the hiding of placeholders altogether though? or just the transition
19:25dmosehttps://wiki.mozilla.org/Firefox/Shield/Shield_Studies#How_Do_I_Request.2FMake_a_Shield_Study
19:26Mardakk88hudson: design says show placeholders
19:32k88hudsonMardak: alright cool, let me just do some testing to makes sure there isn't too much shifting
19:32k88hudsonor not shifting but like
19:33k88hudsonthings being different
19:35dmosek88hudson: Mardak: ursula: mconley is available, join us in "Engagement Services" Vidyo if you can
19:36nanj~"experiment-services" room
20:34Mardakk88hudson: https://reviewboard.mozilla.org/r/188458/diff/2/
21:52streaminatork88hudson: Hey! Someone requested a review: https://github.com/mozilla/activity-stream/pull/3693
22:55streaminatorandreio: Hey! Someone requested a review: https://github.com/mozilla/activity-stream/pull/3694
23:22streaminatorMardak: Hey, your patch was approved! https://github.com/mozilla/activity-stream/pull/3694
12 Oct 2017
No messages
   
Last message: 5 days and 14 hours ago