When I switch repository after doing a local search the text stays in
the search field.
Is this the intended behavior?
Padraig
On 07/10/09 08:13, Padraig O'Briain wrote:
I have no strong views about the change of behavior, but I see that
Jenya is happy with it which is good enough for me.
It is good to see a code change which reduces the size of the code.
I think that you can remove __link_load_blank as it is no longer used;
also the commented out line 1871.
One thing you might take from my original webrev for 9881 is to use
self.w_searchentry.get_text_length() consistently instead of
len(self.w_searchentry.get_text()) or self.w_searchentry.get_text()
== ""
Padraig
On 07/09/09 21:30, jmr wrote:
Hi, this webrev allows filtering of search results using the Show
filter for Installed, Not Installed and so on.
When you go into search now, we do not switch to All Packages. The
Show filter applies to whatever you are looking at.
I have changed how we go into Search All to be more consistent with
Single Search. I have also altered Clear, so now if you backspace in
the Search field to empty it nothing happens to the displayed list,
if you hit Clear it will clear the text field and the list only if
its displaying results. I know this overlaps with a webrev you have
out Padraig for 9881, but think with the other changes its better to
clear the result set and allow the user to search again without
putting them back into Browse mode.
webrev:
http://cr.opensolaris.org/~jmr/pm_filter_search_results_09Jul_810pm/
9944 PM allow filtering of Search Results
9881 Clear and Clear Search no longer restore the full list of packages
JR
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
