Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-09-12 Thread Henri Beauchamp
On Mon, 22 Aug 2011 16:43:22 -, Jonathan Yap wrote: > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/449/ > --- > > Review re

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-27 Thread Vadim ProductEngine
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/#review1000 --- Ship it! - Vadim On Aug. 26, 2011, 9:37 a.m., Jonathan Yap wro

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-26 Thread Jonathan Yap
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/ --- (Updated Aug. 26, 2011, 9:37 a.m.) Review request for Viewer. Changes ---

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-26 Thread Vadim ProductEngine
> On Aug. 26, 2011, 6:05 a.m., Vadim ProductEngine wrote: > > indra/newview/llviewermessage.cpp, lines 6556-6559 > > > > > > It's a hack to update the Block List panel from here. > > The panel should be an observe

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-26 Thread Jonathan Yap
> On Aug. 26, 2011, 6:05 a.m., Vadim ProductEngine wrote: > > indra/newview/lltoastnotifypanel.cpp, line 339 > > > > > > The surrounding code looks quite messy (not your fault, I know), so > > please make more comment

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-26 Thread Vadim ProductEngine
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/#review993 --- indra/newview/lltoastnotifypanel.cpp

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-23 Thread Jonathan Yap
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/ --- (Updated Aug. 23, 2011, 1:41 p.m.) Review request for Viewer. Summary ---

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-23 Thread Jonathan Yap
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/ --- (Updated Aug. 23, 2011, 4:07 a.m.) Review request for Viewer. Changes ---

Re: [opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-23 Thread Vadim ProductEngine
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/#review987 --- These changes don't apply to the v-d tip. Please create a fresh fo

[opensource-dev] Review Request: STORM-1567 Mute button for llDialog popup

2011-08-22 Thread Jonathan Yap
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/449/ --- Review request for Viewer. Summary --- Add a block button to popups fr