script favourites - new features

Big_Noid · 2011-11-15 21:44

Works as expected. only thing I'm missing is an option to clear the current setting.

`Black · 2011-11-15 23:36

I could add it but if you add a button to clear it, you could simply add 3 onclick actions with

Code:

Skin.Reset(property.Label)

Skin.Reset(property.Path)

Skin.Reset(property.Icon)

Or do you want to clear it differently?

@pecinko
No way for the skinner to choose. It's ActivateWindow(...) for everything except videos or songs. User can choose for playlists.

Big_Noid · 2011-11-15 23:42

yeah, but it's much cleaner if it's done like with add-ons. The first item in the list is for "None - clear the current settings". I can't add that in the select dialog that pops-up.
Also if I was to add a clear button for every item then there is even more code than I use now for the same functionality.

pecinko · 2011-11-16 00:00

Big_Noid Wrote:yeah, but it's much cleaner if it's done like with add-ons. The first item in the list is for "None - clear the current settings".

+1 for this if possible

pecinko · 2011-11-16 00:01

`Black Wrote:@pecinko
No way for the skinner to choose. It's ActivateWindow(...) for everything except videos or songs. User can choose for playlists.

If it's like that, that's cool for me.

`Black · 2011-11-16 00:24

Big_Noid Wrote:yeah, but it's much cleaner if it's done like with add-ons. The first item in the list is for "None - clear the current settings". I can't add that in the select dialog that pops-up.
Also if I was to add a clear button for every item then there is even more code than I use now for the same functionality.

Try latest svn. Smile

Sharpe · 2011-11-16 18:18

Hello

Anyone know how I can get this functionality working again?

Code:

RunScript(script.favourites,playlists=window)

I want my playlists to display in a window rather than play (like they used to before these mods).

I saw the bit about a yes/no dialog appearing when you add a playlist as a favourite - so I guess that would do but how does that appear - I'm not seeing it?

#Edit - scratch that figured it out - me being a knob, cheers

`Black · 2011-11-16 18:24

RunScript(script.favourites), if you add playlists, they'll play no matter what string you pass.

Big_Noid · 2011-11-16 18:46

`Black Wrote:Try latest svn.

Now it's perfect. Thx!

I wish you had made this script a week earlier as i just added a dozen custom favorites which i have to rewrite now Tongue

Great work nonetheless Smile

`Black · 2011-11-16 20:31

Thanks

Big_Noid Skilled Skinner Posts: 2,627 Joined: Jul 2010 Reputation: 68 Location: Below sea level	2011-11-15 21:44 Post: #11 Works as expected. only thing I'm missing is an option to clear the current setting.
	find quote

Big_Noid Skilled Skinner Posts: 2,627 Joined: Jul 2010 Reputation: 68 Location: Below sea level	2011-11-15 23:42 Post: #13 yeah, but it's much cleaner if it's done like with add-ons. The first item in the list is for "None - clear the current settings". I can't add that in the select dialog that pops-up. Also if I was to add a clear button for every item then there is even more code than I use now for the same functionality.
	find quote

pecinko Member+ Posts: 3,027 Joined: Dec 2008 Reputation: 27 Location: Prague / Belgrade	2011-11-16 00:00 Post: #14 Big_Noid Wrote:yeah, but it's much cleaner if it's done like with add-ons. The first item in the list is for "None - clear the current settings". +1 for this if possible My skins: Quartz Amber
	find quote

pecinko Member+ Posts: 3,027 Joined: Dec 2008 Reputation: 27 Location: Prague / Belgrade	2011-11-16 00:01 Post: #15 `Black Wrote:@pecinko No way for the skinner to choose. It's ActivateWindow(...) for everything except videos or songs. User can choose for playlists. If it's like that, that's cool for me. My skins: Quartz Amber
	find quote

`Black Skilled Skinner Posts: 2,022 Joined: Apr 2009 Reputation: 50 Location: Germany	2011-11-16 00:24 Post: #16 Big_Noid Wrote:yeah, but it's much cleaner if it's done like with add-ons. The first item in the list is for "None - clear the current settings". I can't add that in the select dialog that pops-up. Also if I was to add a clear button for every item then there is even more code than I use now for the same functionality. Try latest svn.
	find quote

`Black Skilled Skinner Posts: 2,022 Joined: Apr 2009 Reputation: 50 Location: Germany	2011-11-16 18:24 Post: #18 RunScript(script.favourites), if you add playlists, they'll play no matter what string you pass.
	find quote

Big_Noid Skilled Skinner Posts: 2,627 Joined: Jul 2010 Reputation: 68 Location: Below sea level	2011-11-16 18:46 Post: #19 `Black Wrote:Try latest svn. Now it's perfect. Thx! I wish you had made this script a week earlier as i just added a dozen custom favorites which i have to rewrite now Great work nonetheless
	find quote

`Black Skilled Skinner Posts: 2,022 Joined: Apr 2009 Reputation: 50 Location: Germany	2011-11-16 20:31 Post: #20 Thanks
	find quote