Hi Michael, > > looks nice. However you have to break this up into pieces for. Sent > > small patches and I can quickly review and commit them. > > And here's the actual wizard UI patch. It's a bit large, but not > terribly break-down-able. first of all the bluetooth_client_get_name part should be a separate patch. Then the modification to the applet code to show the extra entry can be a separate patch, too. Same goes for the part where you actually enable the installation of the bluetooth-wizard program. Leave these all out, because I don't bother and we will apply them once the UI change is committed. > It adds the passkey code for the UI and modifies the UI a bit, as I > mentioned in previous emails. Lets do the UI changes without the passkey stuff. We should get the wizard working. If no device specific passkey is provided, it will fall back to the applet anyway and that is fine for the first step. I also don't like that you copy the agent.[ch] files. This needs some extra thinking and hence first the UI and then the agent part. > Bastien suggested that the applet label to launch the wizard should > continue to be "Setup" and not "Connect" because we may want to do more > setup for certain device types in the future. That may be true, but > some of that would be automatic (like the SyncML example he cited) and > so wouldn't be visible from the user's perspective. I am for "Setup. Using "Connect" is wrong here. If you wanna connect to a previously setup device, then we have to add shortcuts to the applet menu to do so. Regards Marcel ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Bluez-devel mailing list Bluez-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.sourceforge.net/lists/listinfo/bluez-devel
