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 596294 - Split filter out of chooser
Split filter out of chooser
Status: RESOLVED FIXED
Product: gnome-bluetooth
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-bluetooth-general-maint@gnome.bugs
gnome-bluetooth-general-maint@gnome.bugs
Depends on:
Blocks:
 
 
Reported: 2009-09-25 11:00 UTC by Joshua Lock
Modified: 2009-09-25 15:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add seahorse-bind (from Seahorse) for binding GObject properties (18.01 KB, patch)
2009-09-25 11:00 UTC, Joshua Lock
committed Details | Review
Create an external widget for the device filter (21.29 KB, patch)
2009-09-25 11:01 UTC, Joshua Lock
none Details | Review
Remove filter creation from the BluetoothChooser and use new filter widget (10.82 KB, patch)
2009-09-25 11:01 UTC, Joshua Lock
none Details | Review
Some filter widget fixes (5.60 KB, patch)
2009-09-25 12:38 UTC, Bastien Nocera
none Details | Review
Create a separate widget for the device filter (21.78 KB, patch)
2009-09-25 14:22 UTC, Joshua Lock
none Details | Review
Convert the BluetoothChooser to use BluetoothFilterWidget (13.02 KB, patch)
2009-09-25 14:23 UTC, Joshua Lock
committed Details | Review
Create a separate widget for the device filter (21.73 KB, patch)
2009-09-25 15:43 UTC, Joshua Lock
committed Details | Review

Description Joshua Lock 2009-09-25 11:00:11 UTC
I'd like to have the filter part of the chooser split out into a separate widget so that it can be placed independently of the chooser.

I've begun work on this, and have some patches towards an initial implementation to attach.
Comment 1 Joshua Lock 2009-09-25 11:00:40 UTC
Created attachment 143977 [details] [review]
Add seahorse-bind (from Seahorse) for binding GObject properties

Hopefully we can remove this once GObject integrates such functionality
as tracked in BGO#348080 https://bugzilla.gnome.org/show_bug.cgi?id=348080
Comment 2 Joshua Lock 2009-09-25 11:01:16 UTC
Created attachment 143978 [details] [review]
Create an external widget for the device filter
Comment 3 Joshua Lock 2009-09-25 11:01:28 UTC
Created attachment 143979 [details] [review]
Remove filter creation from the BluetoothChooser and use new filter widget
Comment 4 Joshua Lock 2009-09-25 11:02:36 UTC
I'm submitting this patch for initial review, it doesn't actually all work together as advertised yet. I'm getting:

(lt-test-deviceselection:24590): GLib-GObject-CRITICAL **: g_object_set: assertion `G_IS_OBJECT (object)' failed

But thought I might as well let it all get some review while I debug this.
Comment 5 Bastien Nocera 2009-09-25 12:38:30 UTC
Created attachment 143980 [details] [review]
Some filter widget fixes
Comment 6 Joshua Lock 2009-09-25 14:22:49 UTC
Created attachment 143997 [details] [review]
Create a separate widget for the device filter

This widget will be created by the chooser by default but can instead be
created as a separate widget and associated with a chooser.
Comment 7 Joshua Lock 2009-09-25 14:23:10 UTC
Created attachment 143998 [details] [review]
Convert the BluetoothChooser to use BluetoothFilterWidget
Comment 8 Joshua Lock 2009-09-25 15:43:07 UTC
Created attachment 144012 [details] [review]
Create a separate widget for the device filter

This widget will be created by the chooser by default but can instead be
created as a separate widget and associated with a chooser.
Comment 9 Bastien Nocera 2009-09-25 15:56:32 UTC
Attachment 143977 [details] pushed as 5a8246e - Add seahorse-bind (from Seahorse) for binding GObject properties
Attachment 143998 [details] pushed as df9e79c - Convert the BluetoothChooser to use BluetoothFilterWidget
Attachment 144012 [details] pushed as fd638b1 - Create a separate widget for the device filter