Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-29 Thread Gary Martin
Hi Simon, On 29 Aug 2012, at 12:41, Simon Schampijer wrote: > On 08/28/2012 05:53 PM, Gary Martin wrote: >> Hi Simon, >> >> On 28 Aug 2012, at 14:12, Simon Schampijer wrote: >> >>> On 08/28/2012 02:14 PM, Manuel Quiñones wrote: So now the filtering is shared between the three views. I t

Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-29 Thread Gary Martin
On 29 Aug 2012, at 18:19, Gonzalo Odiard wrote: > I am pretty sure Aleksey knows who was involved :) > > There are more info here [1] and here [2] > > Solve the packaging issues (specially the dependencies required) > is another different beast, but we can add the following without problem: >

Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-29 Thread Gonzalo Odiard
I am pretty sure Aleksey knows who was involved :) There are more info here [1] and here [2] Solve the packaging issues (specially the dependencies required) is another different beast, but we can add the following without problem: summary (will be translated) license homepage And at the risk o

Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-29 Thread Gary Martin
Hi Gonzalo, On 29 Aug 2012, at 13:51, Gonzalo Odiard wrote: > > Btw, re Activity list view. We said at one point that the date displayed in > the list has not much meaning (at least at the moment). Did we have an idea > for a replacement? > > +1 > > I know is not a trivial change, but IMHO,

Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-29 Thread Gonzalo Odiard
> > > Btw, re Activity list view. We said at one point that the date displayed > in the list has not much meaning (at least at the moment). Did we have an > idea for a replacement? > > +1 I know is not a trivial change, but IMHO, would be great have a short description here. We can add it in the a

Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-29 Thread Simon Schampijer
On 08/28/2012 05:53 PM, Gary Martin wrote: Hi Simon, On 28 Aug 2012, at 14:12, Simon Schampijer wrote: On 08/28/2012 02:14 PM, Manuel Quiñones wrote: So now the filtering is shared between the three views. I think this is worth noting in the commit message. Thanks for the review and catch

Re: [Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-28 Thread Gary Martin
Hi Simon, On 28 Aug 2012, at 14:12, Simon Schampijer wrote: > On 08/28/2012 02:14 PM, Manuel Quiñones wrote: >> So now the filtering is shared between the three views. I think this >> is worth noting in the commit message. > > Thanks for the review and catching this! I did note it in the commi

Re: [Sugar-devel] [DESIGN] Views: search behavior

2012-08-28 Thread Sascha Silbe
Simon Schampijer writes: > - if you did a search in View A and then switch to View B the search > from A will not be applied in B, if you switch back to A the search you > did before in A has been cached and is still applied (behavior before > 0.97.2) IMO this makes the most sense to the user

[Sugar-devel] [DESIGN] Views: search behavior (was Re: [PATCH sugar] Views: move the ViewToolbar to the HomeWindow instead of having one in each View)

2012-08-28 Thread Simon Schampijer
On 08/28/2012 02:14 PM, Manuel Quiñones wrote: So now the filtering is shared between the three views. I think this is worth noting in the commit message. Thanks for the review and catching this! I did note it in the commit message before pushing. Gary, we did discuss the following: There