<br><br><div class="gmail_quote">On Tue, Apr 19, 2011 at 2:47 PM, Daniel Drake <span dir="ltr"><<a href="mailto:dsd@laptop.org">dsd@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 15 April 2011 15:06, Gonzalo Odiard <<a href="mailto:godiard@sugarlabs.org">godiard@sugarlabs.org</a>> wrote:<br>
> I have not moved the record buttons yet.<br>
> The general design of the UI is discussed in<br>
> <a href="http://wiki.laptop.org/go/User:Godiard/Record/NewToolbar#Ideas_to_discuss_the_UI_for_Record" target="_blank">http://wiki.laptop.org/go/User:Godiard/Record/NewToolbar#Ideas_to_discuss_the_UI_for_Record</a><br>

<br>
</div>Thanks for working on this.<br>
<br>
I had a look at the results. It is hard to see which mode you are in,<br>
e.g. if you click on video then on the scissors, it's not obvious if<br>
you are in video or audio or photo mode.<br>
<br></blockquote><div><br>Yes, I agree. I am thinking in modifing the buttons to show them like a RadioToolButton.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Also, if you are recording a video and click on the photo icon, the<br>
mode change isn't registered:<br>
1. switch to video icon<br>
2. start recording video<br>
3. switch to camera icon<br>
4. stop recording video<br>
5. click shutter again - it starts recording a video, even though you<br>
are in photo mode<br>
<br></blockquote><div><br>Ok, I need to see this cases.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
As for the code, it looks fine so far except for code that gets<br>
commented out and EditToolbar's copy member doesn't need to be a class<br>
variable.<br>
<br></blockquote><div><br>Ok.<br>I will continue working on this. <br>This is a short week, and Simon is here (in BsAs). I hope, I can continue the next week.<br>Thanks for reviewing it.<br><br>Gonzalo<br><br>
</div></div><br>