<p dir="ltr">This looks great!  Please send a patch and we can review it!<br>
</p>
<br><div class="gmail_quote"><div dir="ltr">On Sun, Feb 21, 2016, 05:29 Utkarsh Tiwari <<a href="mailto:iamutkarshtiwari@gmail.com">iamutkarshtiwari@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello everyone,<br>                          I finally managed to pull through 50% of the downloadManagerIcon integration task. Here is the gif of all what I did till now -> <a href="http://imgur.com/zS0W9dG" target="_blank">http://imgur.com/zS0W9dG</a>  <br>There is still a long way to go. Reviews on this would be helpful :)<br><br>regards,<br>Utkarsh Tiwari<br><br> </div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 13, 2016 at 8:36 PM, Utkarsh Tiwari <span dir="ltr"><<a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I need to update a toolbutton in webtoolbar as the download progresses<br>
in background. Could anyone please guide me how to go about it?<br>
<div><div><br>
On 2/11/16, Utkarsh Tiwari <<a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a>> wrote:<br>
> @James As mentioned, I tried making a blend of 'progressIcon' &<br>
> 'ToolButton' by making some changes to the SugarToolButton code. Here is<br>
> how the download ProgressIcon @50% completion looks like -><br>
> <a href="http://imgur.com/iBg4Ktr" rel="noreferrer" target="_blank">http://imgur.com/iBg4Ktr</a><br>
><br>
> Waiting for your reviews :)<br>
><br>
> On Thu, Feb 11, 2016 at 3:28 PM, Sam P. <sam@sam.today> wrote:<br>
><br>
>> Hello,<br>
>><br>
>> I believe that it passes all the key word and normal arguments to the<br>
>> superclass constructors.<br>
>><br>
>> Out of interest, why are you using a Gtk drawing area as an additional<br>
>> base class?  Every widget (even the tool button) has the draw signal.  Is<br>
>> this what you are using it for or is there something else?<br>
>><br>
>> Thanks,<br>
>> Sam<br>
>><br>
>> On Thu, Feb 11, 2016, 09:03 Utkarsh Tiwari <<a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a>><br>
>> wrote:<br>
>><br>
>>> What does the line "GObject.GObject.__init__(self, **kwargs)" exactly<br>
>>> does in the following gist?<br>
>>><br>
>>> <a href="https://gist.github.com/iamutkarshtiwari/5024762d0e8924b52085" rel="noreferrer" target="_blank">https://gist.github.com/iamutkarshtiwari/5024762d0e8924b52085</a><br>
>>><br>
>>> On Thu, Feb 4, 2016 at 5:00 AM, James Cameron <<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>> wrote:<br>
>>><br>
>>>> Thanks.  Yes, the position is as I described.<br>
>>>><br>
>>>> Your question puzzles me.  I didn't think there was any ambiguity or<br>
>>>> doubt.  It is unfortunate that I'm the only one who has posted about<br>
>>>> the positioning of this design change.<br>
>>>><br>
>>>> Is it because you have an alternate suggestion?<br>
>>>><br>
>>>> Good to hear combining the widgets worked.  Looking forward to seeing<br>
>>>> a patch or pull request.<br>
>>>><br>
>>>> On Thu, Feb 04, 2016 at 03:49:54AM +0530, Utkarsh Tiwari wrote:<br>
>>>> > @James Is this the icon position you were talking about as shown in<br>
>>>> > the<br>
>>>> > attachment below? Your hint for using ProgressIcon alongwith<br>
>>>> Toolbutton worked!<br>
>>>> > ;)<br>
>>>> ><br>
>>>> > On Tue, Feb 2, 2016 at 2:17 AM, James Cameron <[1]<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>><br>
>>>> wrote:<br>
>>>> ><br>
>>>> >     A ToolButton normally has an Icon.<br>
>>>> ><br>
>>>> >     Instead of the Icon, assign the ProgressIcon?<br>
>>>> ><br>
>>>> >     As nobody else may have done this, you may need to find and fix<br>
>>>> > any<br>
>>>> >     bugs in ProgressIcon or ToolButton that prevent it.<br>
>>>> ><br>
>>>> >     On Mon, Feb 01, 2016 at 11:31:28PM +0530, Utkarsh Tiwari wrote:<br>
>>>> >     > Is there any way to use a ProgressIcon as a Toolbutton ?<br>
>>>> >     ><br>
>>>> >     > On 2/1/16, Sam P. <sam@sam.today> wrote:<br>
>>>> >     > > Hello!<br>
>>>> >     > ><br>
>>>> >     > > Maybe just a large down arrow and fill it vertically?  See<br>
>>>> >     > > 'emblem-downloads' or 'transfer-from' in sugar-artwork.<br>
>>>> >     > ><br>
>>>> >     > > Thanks,<br>
>>>> >     > > Sam<br>
>>>> >     > ><br>
>>>> >     > ><br>
>>>> >     > > On Mon, Feb 1, 2016 at 9:15 AM, Utkarsh Tiwari <[2]<br>
>>>> >     <a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a>><br>
>>>> >     > > wrote:<br>
>>>> >     > ><br>
>>>> >     > >> How should the Icon for "Download Button" look like? Could<br>
>>>> you please<br>
>>>> >     > >> provide me some artwork for reference. ?<br>
>>>> >     > >><br>
>>>> >     > >> On 2/1/16, Utkarsh Tiwari <[3]<a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a>><br>
>>>> wrote:<br>
>>>> >     > >> > Omg! I have been looking all over internet on how to<br>
>>>> integrate a<br>
>>>> >     > >> > Gtk.ProgressBar into Gtk.Toolbutton and I even tried using<br>
>>>> >     > >> > Gtk.CssProvider styler to scale down the progressbar to a<br>
>>>> button :D<br>
>>>> >     > >> > Now that(ProgressIcon.py) sounds really cool!<br>
>>>> >     > >> ><br>
>>>> >     > >> > Thank you :) You made my work a bit easier ;)<br>
>>>> >     > >> ><br>
>>>> >     > >> > On 2/1/16, James Cameron <[4]<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>> wrote:<br>
>>>> >     > >> >> ProgressBar from Gtk is overkill, we already have a<br>
>>>> ProgressIcon<br>
>>>> >     > >> >> in Sugar.<br>
>>>> >     > >> >><br>
>>>> >     > >> >> In sugar-toolkit-gtk3:<br>
>>>> >     > >> >><br>
>>>> >     > >> >> Look at progressicon.py, you will see it does not use a<br>
>>>> >     ProgressBar,<br>
>>>> >     > >> >> but has a __draw_cb method connected to the "draw"<br>
>>>> > signal.<br>
>>>> >     > >> >><br>
>>>> >     > >> >> Look at toolbutton.py, you will see it wraps ToolButton<br>
>>>> do_draw<br>
>>>> >     > >> >> method.<br>
>>>> >     > >> >><br>
>>>> >     > >> >> On Mon, Feb 01, 2016 at 12:10:25AM +0530, Utkarsh Tiwari<br>
>>>> wrote:<br>
>>>> >     > >> >>> "The button to be a blend of ProgressIcon and<br>
>>>> ToolButton". How to<br>
>>>> >     > >> >>> convert a Gtk.ProgressBar to a Browse ToolButton?<br>
>>>> >     > >> >>><br>
>>>> >     > >> >>> On 1/27/16, James Cameron <[5]<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>> wrote:<br>
>>>> >     > >> >>> > No.<br>
>>>> >     > >> >>> ><br>
>>>> >     > >> >>> > The position marked by your cursor seems to be inside<br>
>>>> the URL<br>
>>>> >     entry<br>
>>>> >     > >> >>> > box.<br>
>>>> >     > >> >>> ><br>
>>>> >     > >> >>> > I said between the URL entry box and the back button.<br>
>>>> >     > >> >>> ><br>
>>>> >     > >> >>> > Review the Browse source code that defines these<br>
>>>> objects.<br>
>>>> >     > >> >>> ><br>
>>>> >     > >> >>> > On Wed, Jan 27, 2016 at 11:39:03AM +0530, Utkarsh<br>
>>>> Tiwari wrote:<br>
>>>> >     > >> >>> >> Is this the exact expected location you were talking<br>
>>>> about the<br>
>>>> >     > >> >>> >> 'download button' as shown in the screenshot below?<br>
>>>> >     > >> >>> >><br>
>>>> >     > >> >>> >> [6]<a href="http://imgur.com/T0hirTZ" rel="noreferrer" target="_blank">http://imgur.com/T0hirTZ</a><br>
>>>> >     > >> >>> >><br>
>>>> >     > >> >>> >> On 1/27/16, James Cameron <[7]<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>><br>
>>>> > wrote:<br>
>>>> >     > >> >>> >> > Thanks for working on this.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > Our Sugar human interface guidelines and existing<br>
>>>> code base<br>
>>>> >     > >> >>> >> > don't<br>
>>>> >     > >> >>> >> > have<br>
>>>> >     > >> >>> >> > decorated floating windows or window management.<br>
>>>> Your<br>
>>>> >     proposal<br>
>>>> >     > >> >>> >> > would<br>
>>>> >     > >> >>> >> > not scale to multiple downloads.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > You might ask for design suggestions on this<br>
>>>> > mailing<br>
>>>> list.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > I've resisted making design suggestions on this<br>
>>>> problem<br>
>>>> >     because<br>
>>>> >     > >> >>> >> > I<br>
>>>> >     > >> >>> >> > don't want to constrain the good ideas that may<br>
>>>> > come<br>
>>>> up.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > To get the design conversation started ...<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > When the first download begins, show a button at<br>
>>>> > the<br>
>>>> top<br>
>>>> >     between<br>
>>>> >     > >> >>> >> > the<br>
>>>> >     > >> >>> >> > URL entry box and the back button.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > The button to be a blend of ProgressIcon and<br>
>>>> ToolButton, with<br>
>>>> >     > >> >>> >> > the<br>
>>>> >     > >> >>> >> > progress fill to show average progress of all<br>
>>>> downloads.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > When the button is pressed a download tab is to be<br>
>>>> shown in<br>
>>>> >     the<br>
>>>> >     > >> >>> >> > canvas, which lists downloads and progress.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > The download tab can be closed like any other tab.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > The button is to be be hidden before any downloads<br>
>>>> are<br>
>>>> >     started,<br>
>>>> >     > >> >>> >> > shown<br>
>>>> >     > >> >>> >> > on start of first download, and only hidden if the<br>
>>>> download<br>
>>>> >     tab<br>
>>>> >     > >> >>> >> > is<br>
>>>> >     > >> >>> >> > closed when there are no downloads still active.<br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > On Tue, Jan 26, 2016 at 06:08:32PM +0530, Utkarsh<br>
>>>> Tiwari<br>
>>>> >     wrote:<br>
>>>> >     > >> >>> >> >> Hi,<br>
>>>> >     > >> >>> >> >><br>
>>>> >     > >> >>> >> >>      I am trying to integrate the "Download<br>
>>>> ProgressBar" in<br>
>>>> >     the<br>
>>>> >     > >> >>> >> >> download<br>
>>>> >     > >> >>> >> >> manager. Here is the screenshot of the prototype<br>
>>>> > on<br>
>>>> which I<br>
>>>> >     am<br>
>>>> >     > >> >>> >> >> currently<br>
>>>> >     > >> >>> >> >> working -> [1][8]<a href="http://imgur.com/WUJL8eX" rel="noreferrer" target="_blank">http://imgur.com/WUJL8eX</a><br>
>>>> >     > >> >>> >> >><br>
>>>> >     > >> >>> >> >> Since the download progress bar is popping over<br>
>>>> > the<br>
>>>> browse<br>
>>>> >     > >> >>> >> >> activity,<br>
>>>> >     > >> >>> >> >> is<br>
>>>> >     > >> >>> >> >> it<br>
>>>> >     > >> >>> >> >> feasible?<br>
>>>> >     > >> >>> >> >><br>
>>>> >     > >> >>> >> >> regards,<br>
>>>> >     > >> >>> >> >> Utkarsh Tiwari<br>
>>>> >     > >> >>> >> >><br>
>>>> >     > >> >>> >> >> References:<br>
>>>> >     > >> >>> >> >><br>
>>>> >     > >> >>> >> >> [1] [9]<a href="http://imgur.com/WUJL8eX" rel="noreferrer" target="_blank">http://imgur.com/WUJL8eX</a><br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> >> _______________________________________________<br>
>>>> >     > >> >>> >> >> Sugar-devel mailing list<br>
>>>> >     > >> >>> >> >> [10]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
>>>> >     > >> >>> >> >><br>
>>>> > [11]<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> >> > --<br>
>>>> >     > >> >>> >> > James Cameron<br>
>>>> >     > >> >>> >> > [12]<a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> >     > >> >>> >> > _______________________________________________<br>
>>>> >     > >> >>> >> > Sugar-devel mailing list<br>
>>>> >     > >> >>> >> > [13]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
>>>> >     > >> >>> >> > [14]<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
>>>> >     > >> >>> >> ><br>
>>>> >     > >> >>> ><br>
>>>> >     > >> >>> > --<br>
>>>> >     > >> >>> > James Cameron<br>
>>>> >     > >> >>> > [15]<a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> >     > >> >>> ><br>
>>>> >     > >> >><br>
>>>> >     > >> >> --<br>
>>>> >     > >> >> James Cameron<br>
>>>> >     > >> >> [16]<a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> >     > >> >><br>
>>>> >     > >> ><br>
>>>> >     > >> _______________________________________________<br>
>>>> >     > >> Sugar-devel mailing list<br>
>>>> >     > >> [17]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
>>>> >     > >> [18]<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
>>>> >     > >><br>
>>>> >     > ><br>
>>>> ><br>
>>>> >     --<br>
>>>> >     James Cameron<br>
>>>> >     [19]<a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> ><br>
>>>> > References:<br>
>>>> ><br>
>>>> > [1] mailto:<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a><br>
>>>> > [2] mailto:<a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a><br>
>>>> > [3] mailto:<a href="mailto:iamutkarshtiwari@gmail.com" target="_blank">iamutkarshtiwari@gmail.com</a><br>
>>>> > [4] mailto:<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a><br>
>>>> > [5] mailto:<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a><br>
>>>> > [6] <a href="http://imgur.com/T0hirTZ" rel="noreferrer" target="_blank">http://imgur.com/T0hirTZ</a><br>
>>>> > [7] mailto:<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a><br>
>>>> > [8] <a href="http://imgur.com/WUJL8eX" rel="noreferrer" target="_blank">http://imgur.com/WUJL8eX</a><br>
>>>> > [9] <a href="http://imgur.com/WUJL8eX" rel="noreferrer" target="_blank">http://imgur.com/WUJL8eX</a><br>
>>>> > [10] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
>>>> > [11] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
>>>> > [12] <a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> > [13] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
>>>> > [14] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
>>>> > [15] <a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> > [16] <a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>> > [17] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
>>>> > [18] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
>>>> > [19] <a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>><br>
>>>><br>
>>>><br>
>>>> --<br>
>>>> James Cameron<br>
>>>> <a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
>>>><br>
>>><br>
>>><br>
><br>
</div></div></blockquote></div><br></div>
</blockquote></div>