<div dir="ltr"><div>I updated my patch to use a ComboBox.<br></div><div>Same link: <a href="https://github.com/edudev/sugar/compare/ticket3026.patch">https://github.com/edudev/sugar/compare/ticket3026.patch</a><br><br></div>
<div>Since not using a slider, the class I initially create is unneeded. The only thing I don't like in the code is the count of the global variables...<br><br></div><div>Not sure how to tweak the time spans.<br><br></div>
<div>Emil Dudev<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 3, 2014 at 10:50 PM, Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org" target="_blank">manuq@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2013/12/4 Daniel Narvaez <<a href="mailto:dwnarvaez@gmail.com">dwnarvaez@gmail.com</a>>:<br>
<div class="im">> The tag seems to be helpful to get designers attention :)<br>
><br>
> On Tuesday, 3 December 2013, Walter Bender wrote:<br>
>><br>
>> Emil's code works well; the question to me is one of design. He<br>
>> replaced the comboboxes with a slider. I wonder if perhaps a palette<br>
>> (like the one Gonzalo developed for Fonts) might be better. We also<br>
>> may want to tweak the actually times made available. But Emil's patch<br>
>> does capture the spirit of what we are trying to accomplish.<br>
<br>
</div>Good point Walter.  At first glance, I think the slider does not fit<br>
the model.  I can't see why:<br>
<br>
1. this week < 2. last week < 3. this month<br>
<br>
Having a journal entry made today, I can see it in 1. and 3, and not in 2.<br>
<br>
So I think a combo, like it is now, fits better.<br>
<br>
Side note: I have applied the patch<br>
(<a href="https://github.com/edudev/sugar/compare/ticket3026.patch" target="_blank">https://github.com/edudev/sugar/compare/ticket3026.patch</a>), but I'm<br>
having a hard time testing it with a new sugar build, just because i<br>
don't have old journal entries.  So I guess I'll have to create mock<br>
entries for testing this.  Has anyone did this before?  Or do we have<br>
a sample datastore for testing?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
.. manuq ..<br>
</font></span></blockquote></div><br></div>