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 705089 - Search box eats first character after collections click, clears selection
Search box eats first character after collections click, clears selection
Status: RESOLVED FIXED
Product: gnome-packagekit
Classification: Core
Component: general
3.8.x
Other Linux
: Normal minor
: ---
Assigned To: gnome-packagekit-maint
gnome-packagekit-maint
Depends on:
Blocks:
 
 
Reported: 2013-07-29 16:58 UTC by Richard McNab
Modified: 2014-06-29 16:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (2.13 KB, patch)
2013-08-23 15:17 UTC, Ignazio Sgalmuzzo
committed Details | Review

Description Richard McNab 2013-07-29 16:58:58 UTC
Searching after opening packagekit is ok.
But then click a collection- the contents show properly, but if you then click to search, the first character you type will disappear, and the selected collection will be cleared.
Comment 1 Ignazio Sgalmuzzo 2013-08-23 15:17:52 UTC
Created attachment 252878 [details] [review]
patch

Here'a a patch for the first character eating.

It does not change the behaviour when text is entered in the search entry (the package group is deselected and, accordingly, the package list is emptied): looking at the code it seems to me it was the intended behaviour.
Comment 2 Michael Catanzaro 2014-06-29 02:29:22 UTC
Review of attachment 252878 [details] [review]:

Thanks for taking the time to work on a small bug, and sorry it took a while to get a review.

This works fine, and looks fine except for a style nit.

::: src/gpk-application.c
@@ +1767,3 @@
 	/* clear group selection if we have the tab */
+	if (pk_bitfield_contain (priv->roles, PK_ROLE_ENUM_SEARCH_GROUP) &&
+	    gtk_entry_get_text_length(entry)>0) {

Can you add a space in front of the opening parenthesis before entry, and one on both sides of the >
Comment 3 Michael Catanzaro 2014-06-29 16:50:17 UTC
(In reply to comment #2)
> Can you add a space in front of the opening parenthesis before entry, and one
> on both sides of the >

Really petty of me to ask you to fix something this trivial after so long a wait; I'm just going to fix and push.