[sugar] [PATCH] Add indications for empty journal and empty search results

Eben Eliason eben.eliason at gmail.com
Tue Jun 17 20:46:18 EDT 2008


A cleaner attempt.  Thanks.

- Eben


On Tue, Jun 17, 2008 at 11:08 AM, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
> On Tue, Jun 17, 2008 at 4:33 PM, Eben Eliason <eben.eliason at gmail.com> wrote:
>> On Tue, Jun 17, 2008 at 9:18 AM, Eben Eliason <eben.eliason at gmail.com> wrote:
>>> On Tue, Jun 17, 2008 at 3:36 AM, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
>>>> What about creating the empty_box in a method? We don't really need to
>>>> hold a reference to it in ListView, right? You could then pass the
>>>> string as a parameter to that method, thus avoiding multiple repeated
>>>> instances of translatable strings, that is something quite bad.
>>>
>>> We could use a method.  I want to keep the door open to allowing other
>>> changes apart from the string, in the future.  For instance, as the
>>> commit message indicates, the different cases may have buttons with
>>> different icons/actions in the future.  Perhaps I should just make
>>> separate methods for the two states. What do you think?  Perhaps add
>>> another to clear the message and restore the list as well?
>>
>> Oh, as a side note, implementing this as a single method doesn't take
>> care of the repeated translatable strings, as we'd need to pass the
>> translated string from each location we called the method from.  I
>> suppose you could store a reference to the translated strings in init
>> or something, and then pass those, but that doesn't seem like a good
>> solution to me.
>
> Well, being a separate method or class allows you to add a constant
> parameter/property that signals to use one or the other string.
>
> Thanks,
>
> Tomeu
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-indications-for-empty-journal-and-empty-search-r.patch
Type: text/x-diff
Size: 4979 bytes
Desc: not available
Url : http://lists.laptop.org/pipermail/sugar/attachments/20080617/ed1de303/attachment.patch 


More information about the Sugar mailing list