← Back to team overview

ayatana-commits team mailing list archive

Re: [Merge] lp:~c10ud/indicator-application/runtime-themepath-change into lp:indicator-application

 

What do you guys think about changing the signal to include the path in it, that's save a DBus call.  From:

+		<signal name="NewIconThemePath">
+	    </signal>

To:

<signal name="NewIconThemePath">
<arg type="s" name="icon_theme_path" direction="out" />
</signal>

The others don't have properties becasue they could represent a few values changing, but this signal seems to be 1:1 with the theme path, so I'm thinking just send it.

Thoughts?
-- 
https://code.launchpad.net/~c10ud/indicator-application/runtime-themepath-change/+merge/30695
Your team ayatana-commits is subscribed to branch lp:indicator-application.



References