! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Clearing search leaves Settings missing sections
Status:
RESOLVED: FIXED
Product:
Xfce4-settings
Component:
General

Comments

Description flocculant 2018-10-23 06:30:24 CEST
Created attachment 8059 
video

Hard to word this.

Search in Settings and backspace to clear the search leaves many setting options missing.

Search in Setttings and use the clear search button leaves Settings with an unusable list.

In both methods the return to All Settings is disabled.

Attached small video.
Comment 1 flocculant 2018-10-23 06:44:34 CEST
Ignore the all settings comment ... 

Works in 4.12.4
Comment 2 Simon Steinbeiss editbugs 2018-10-23 19:59:29 CEST
Strange, I can't reproduce this with 4.13.5.
Comment 3 Simon Steinbeiss editbugs 2018-10-23 20:23:32 CEST
Ok sorry, actually now I can reproduce it.

As brainwash mentioned on IRC, if the settings manager window loses focus the correct content is restored.
Comment 4 Andre Miranda editbugs 2019-01-17 23:29:18 CET
*** Bug 15054 has been marked as a duplicate of this bug. ***
Comment 5 Theo Linkspfeifer editbugs 2019-04-20 14:27:12 CEST
https://git.xfce.org/xfce/xfce4-settings/tree/xfce4-settings-manager/xfce-settings-manager-dialog.c#n732

Removing or changing this line to gtk_widget_set_sensitive() seems to resolve the bug.
Comment 6 Simon Steinbeiss editbugs 2019-04-21 00:21:18 CEST
That's a good workaround, I agree. I'm pretty sure the bug is within ExoIconView though.

@Sean: What's your take? Shall we
1) push the workaround and port the dialog away from ExoIconView in the future?
2) push the workaround and fix the bug in ExoIconView?
3) not push the workaround and try to fix the bug in ExoIconView immediately?
Comment 7 Simon Steinbeiss editbugs 2019-04-21 00:40:19 CEST
There's actually a downside to the workaround, it will make the search slower/more expensive, because only visible categories are searched.
Comment 8 Theo Linkspfeifer editbugs 2019-04-24 12:38:51 CEST
Created attachment 8453 
diff

Instead of checking the visibility of a category we could check the widget’s sensitivity.
Comment 9 Simon Steinbeiss editbugs 2019-04-25 17:53:02 CEST
Created attachment 8456 
Screenshot after clearing the search
Comment 10 Simon Steinbeiss editbugs 2019-04-25 17:56:37 CEST
The bug isn't actually gone with the _set_sensitive workaround, it just appears less often (see screenshot).

I can still reproduce it easily following these steps:
1) type in "xterm" (so no entries are left)
2) clear the search entry
3) type in "xterm" again
4) result is broken categories while deleting single characters plus not all items shown when the entry is cleared
Comment 11 Simon Steinbeiss editbugs 2019-05-04 23:18:30 CEST
As far as I can tell this bug is in exo, more concretely in exo_icon_view_layout.
I'm not sure why the calculations there are off but I suspect it's gtk_widget_get_preferred_size (exo icon view uses the natural over the minimum size, not sure what that results in) as that has been a troublemaker in other components too...
Comment 12 Simon Steinbeiss editbugs 2019-05-04 23:34:09 CEST
(The hint could be this FIXME https://git.xfce.org/xfce/exo/tree/exo/exo-icon-view.c#n9233)
Comment 13 Git Bot editbugs 2019-05-06 00:12:02 CEST
Sean Davis referenced this bugreport in commit 187eee6d9d3d41404b5c6f1fc56510f780afd703

Fix search change clearing and hiding sections (bug #14792)

https://git.xfce.org/xfce/xfce4-settings/commit?id=187eee6d9d3d41404b5c6f1fc56510f780afd703

Bug #14792

Reported by:
flocculant
Reported on: 2018-10-23
Last modified on: 2019-05-09
Duplicates (1):

People

Assignee:
Sean Davis
CC List:
6 users

Version

Version:
4.13.5

Attachments

video (774.21 KB, video/x-matroska)
2018-10-23 06:30 CEST , flocculant
no flags
diff (1.87 KB, patch)
2019-04-24 12:38 CEST , Theo Linkspfeifer
no flags
Screenshot after clearing the search (50.81 KB, image/png)
2019-04-25 17:53 CEST , Simon Steinbeiss
no flags

Additional information