After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 128109 - Add "Clear" button to reset the search entry
Add "Clear" button to reset the search entry
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: User Interface
HEAD
Other Linux
: Normal enhancement
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 131151 135553 138952 139892 319892 319893 319902 326423 335174 342128 343658 (view as bug list)
Depends on:
Blocks: 326423
 
 
Reported: 2003-11-28 12:03 UTC by Hakon
Modified: 2006-06-02 02:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
rb-view-all-menu-item.patch (4.53 KB, patch)
2004-05-02 23:59 UTC, Bastien Nocera
none Details | Review
Uploaded patch provided by James "Doc" Livingston on the ML (29.82 KB, patch)
2006-03-20 02:37 UTC, Alex Lancaster
none Details | Review
updated patch (30.70 KB, patch)
2006-04-02 03:47 UTC, James "Doc" Livingston
committed Details | Review

Description Hakon 2003-11-28 12:03:37 UTC
Search field should have a Clear button after it (like in Evolution) so
that you could easily reset the view to non-filtered once you have found
what you looked for.
Comment 1 Bastien Nocera 2003-11-28 12:12:34 UTC
We already discussed this on the ML (see:
http://lists.gnome.org/archives/rhythmbox-devel/2003-November/msg00189.html)

It was agreed that we should add an entry in the View menu saying
"View All" which would reset the filters and the search entry.
Comment 2 Bastien Nocera 2004-02-26 23:45:13 UTC
*** Bug 135553 has been marked as a duplicate of this bug. ***
Comment 3 Bastien Nocera 2004-02-26 23:46:00 UTC
*** Bug 131151 has been marked as a duplicate of this bug. ***
Comment 4 Bastien Nocera 2004-04-05 14:07:21 UTC
*** Bug 138952 has been marked as a duplicate of this bug. ***
Comment 5 Bastien Nocera 2004-04-13 09:14:49 UTC
*** Bug 139892 has been marked as a duplicate of this bug. ***
Comment 6 Bastien Nocera 2004-05-02 23:59:45 UTC
Created attachment 27314 [details] [review]
rb-view-all-menu-item.patch

Patch, as discussed.
I'm not sure which kbd shortcut we should use.
Furthermore, the search stuff is incredibly twisted, and I didn't manage to
make the "View All" menu unsensitive when the "search" field is empty.
Hopefully you will ;)
Comment 7 Colin Walters 2004-05-05 23:28:36 UTC
Ok, I'm going to put this patch off for a bit, as I have a branch which is
converted to use GtkAction.  Once I merge that let's have a look at the patch.
Comment 8 Colin Walters 2004-07-15 03:55:42 UTC
Finally got a chance to look this over and port it to the new GtkAction bits.  I
also added a keybinding at the same time.  I chose "Ctrl+Y", because very little
remains free :/  If you think of something better let me know...

Anyways,
* committed rhythmbox-devel@gnome.org--2004/rhythmbox--main--0.9--patch-9

Thanks Bastien!
Comment 9 Bastien Nocera 2005-10-26 20:10:52 UTC
*** Bug 319892 has been marked as a duplicate of this bug. ***
Comment 10 Bastien Nocera 2005-10-26 20:11:15 UTC
*** Bug 319893 has been marked as a duplicate of this bug. ***
Comment 11 Bastien Nocera 2005-10-26 21:54:51 UTC
*** Bug 319902 has been marked as a duplicate of this bug. ***
Comment 12 Jonathan Matthew 2006-02-12 20:23:49 UTC
*** Bug 326423 has been marked as a duplicate of this bug. ***
Comment 13 Alex Lancaster 2006-03-20 00:47:29 UTC
*** Bug 335174 has been marked as a duplicate of this bug. ***
Comment 14 Alex Lancaster 2006-03-20 00:51:53 UTC
Given new interface and discussion on the mailing list, see:

http://mail.gnome.org/archives/rhythmbox-devel/2006-March/msg00035.html

I propose that this bug should be re-opened (/me ducks) since it is clearly a highly requested feature that could be implemented fairly unobtrusively, see also:

http://mail.gnome.org/archives/rhythmbox-devel/2006-March/msg00056.html
Comment 15 Alex Lancaster 2006-03-20 02:37:13 UTC
Created attachment 61584 [details] [review]
Uploaded patch provided by James "Doc" Livingston on the ML

Patch by "Doc" from here:

http://mail.gnome.org/archives/rhythmbox-devel/2006-March/binS5UIeMVHfB.bin

linked from this post:

http://mail.gnome.org/archives/rhythmbox-devel/2006-March/msg00056.html

Tested this patch on FC-4 with libsexy-devel package from Fedora Extras and it works well.
Comment 16 Alex Lancaster 2006-03-20 21:41:55 UTC
Re-opening bug after discussing it with James Livingston on IRC.
Comment 17 Alex Lancaster 2006-03-20 21:44:13 UTC
Retitling summary.
Comment 18 James "Doc" Livingston 2006-04-02 03:47:53 UTC
Created attachment 62582 [details] [review]
updated patch

This patch allows building against a system-wide copy of libsexy, but will use an internal copy if it can't be found or --with-internal-libsexy is found.
Comment 19 Alex Lancaster 2006-04-02 09:45:43 UTC
(In reply to comment #18)
> Created an attachment (id=62582) [edit]
> updated patch
> 
> This patch allows building against a system-wide copy of libsexy, but will use
> an internal copy if it can't be found or --with-internal-libsexy is found.

Patch works for me. 

Comment 20 James "Doc" Livingston 2006-04-09 02:04:21 UTC
Committed to cvs.
Comment 21 Andrew Conkling 2006-04-12 03:42:10 UTC
(In reply to comment #8)
> Finally got a chance to look this over and port it to the new GtkAction bits.  I
> also added a keybinding at the same time.  I chose "Ctrl+Y", because very little
> remains free :/  If you think of something better let me know...

Looks like this thread is going in a different direction, but FWIW I think Ctrl+Shift+Backspace would be good (in addition to Ctrl+Y?) since it is easy to remember (for me) as akin to GTK+'s Ctrl+Backspace.
Comment 22 Jonathan Matthew 2006-05-17 14:51:25 UTC
*** Bug 342128 has been marked as a duplicate of this bug. ***
Comment 23 James "Doc" Livingston 2006-06-02 02:09:20 UTC
*** Bug 343658 has been marked as a duplicate of this bug. ***