@eyedeekay
&eche|on
&zzz
+Irc2PGuest81845
+R4SAS
+RN
+StormyCloud
+acetone
+cumlord
+dr|z3d
+eche|off
+mareki2p
+orignal
+postman
+qend-irc2p
+snex
Arch2
Birdy
BubbRubb
Chrono
Daddy
Dann
DeltaOreo
Irc2PGuest27269
Irc2PGuest304
Irc2PGuest53924
Irc2PGuest65801
Irc2PGuest74254
Irc2PGuest90985
Irc2PGuest92550
Irc2PGuest95708
Onn4l7h
Onn4|7h
Over
Sisyphus
Sleepy
SlippyJoe
Stormycloud_
T3s|4_
Teeed
aargh2
ac9f_
anontor
b3t4f4c3__
dr4wd3
duanin2
duck
eyedeekay_bnc
gellegery
makoto
matean
n1
n1_
nilbog
not_bob_afk
null912
onon_
poriori
profetikla
r00tobo_BNC
shiver_
solidx66
thetia
u5657
uop23ip
w8rabbit
x74a6
xHarr
zzz
ping eyedeekay
eyedeekay
pong zzz
zzz
eyedeekay, github 3 weeks behind
eyedeekay
Restarted the sync script, should be good in a moment
zzz
thanks
zzz
eyedeekay, re: MRs
zzz
what goes in for the release and when is up to you
zzz
but if you want any of the things on my list you need to pick up the review pace or we're going to run out of time
zzz
what I've posted so far is maybe a quarter of what I'm sitting on
eyedeekay
Ack I'll focus up on the existing ones today
zzz
and ditto for whatever you're sitting on, nothing from you so far
zzz
vacation's over, let's get to work :)
eche|off
vacation? no vacations available...
dr|z3d
last 3 MRs from zzz are probably the easiest the wave through, I'm running 2/3 without issue.
dr|z3d
also, don't forget snark, zzz, re iframe-resizer. that also benefits.
dr|z3d
with iframe-resizer deployed, you can ship snark with the add/create sections collapsed, as intended.
dr|z3d
it'll also benefit the ajax search, because that changes the height of the page when activated.
zzz
yeah eyedeekay if you have q's on any of them ask here, drz may have the answers
zzz
so the plan is a -3 tonight or tomorrow with whatever gets approved
zzz
and then we'll go around again with the next batch
dr|z3d
I don't think you're using the data-iframe-height attribute to set the child window height, zzz; I found that the most reliable method.
dr|z3d
*to set the container iframe height, sorry.
dr|z3d
also, re sidebar comments in your MR notes, unless I'm mistaken, I think we did away with the iframe container a long time ago.
zzz
it wasn't necessary for the 4 embeds, but it is necessary for susimail html emails, in a MR to come
dr|z3d
to collapse the snark sections by default, just toggle the default checkbox state.
eyedeekay
nack for now on advanced config splitting, dark theme is broke, I'll suggest some changes tonight
eyedeekay
That's probably all the review I'll reasonably get to today, got a couple other things on my docket still
zzz
agreed, I don't think that one is ready