Skin Settings under Addon Setting?

Hitcher Wrote:To achieve this couldn't we have a new dialog - DialogSkinSettings?

I thought it's a skinners choice - I'm displaying addon setting in full screen.

spiff Wrote:it would work EXACTLY as it does for other add-ons now - that's the whole point........... (add some more dots)

it's not our job to protect retards against themself. and you have to be truely retarded to not understand that you're configuring the skin you selected in the add-on browser.

spiff,

Have to say that I'm almost entirely in agreement here with you. But having seen more than one or two posts go thru complaining that 'files' mode has gone, I can see this being more of an issue than 'files' mode.

Just asking that people remember that idiots are incredibly resourceful in doing stupid sh!t. Skins are not scripts and plugins, so need a slightly different mindset. I think the idea of coming off of the appearance setting menu is a good one.

But that is just my 2c worth
Wyrm (xTV-SAF)

sure, that is very much doable, i never imagined anything but a shortcut in the settings (and it's already supported actually - addon.default.OpenSettings(xbmc.skin) (don't try this in current master, it will likely fail horribly). but i don't see how anyone can get confused which skin they are configuring, since you will have to go through the browser to configure anything but the currently used one, that was my point.

pecinko Wrote:I thought it's a skinners choice - I'm displaying addon setting in full screen.

Yes but I'd like to keep the general Addon Settings as a popup and have Skin Settings fullscreen. That way there's no difference to the user between the two methods.

EDIT: And if you wanted them to look the same you could just C&P most of DialogAddonSettings to DialogSkinSettings.

Why do you want skin settings full screen but other addons not full screen?

Some addons have lots and lots of settings (youtube for instance) - some more than some skins have.

Either way I think you could achieve a fullscreen look in a dialog for your skin settings via a conditional include?

I'd like to bring this up to the table again. I already done some work on using addon mechanism to store settings via addon mechanism (this change *should* transparent for skinners).

What type of settings do You need to drop custom window for skin settings in favour of addon settings dialog? Check pdf http://code.google.com/p/xbmc-gpodder-in...loads/list (page 11) to get list of currently available type of settings.

What for sure is missing now is ability to set addon (already have pull request on that) and [optional] ability to display selected image/multiimage for backdrop selector.

What are benefits? You don't need to maintain yet another window and just use simplified addon settings dialog. You can specify default values (f.e. proper bool defaults without weird inverts, set default backdrops)

And once again - You still we able to use custom window for settings.

Only objection I have against this is that it's up to the skinner to decide where to place the settings. Not everybody will move the settings to the add-on settings or will have additional settings windows. This can be confusing to the end-user imo.

Surely the end-user wont know the difference?

Selecting 'Add-on Settings' will either take you a custom Skin Settings window (if present) or Add-on Settings.

Or selecting 'Skin Settings' from Appearance will take you to either the Skin Settings window or Add-on Settings.

I missed page 2 there...
How will that work, do we get a new window id for the skin settings adn if that is used that window will pop-up and otherwise add-on settings screen?

pieh Wrote:I'd like to bring this up to the table again. I already done some work on using addon mechanism to store settings via addon mechanism (this change *should* transparent for skinners).

What type of settings do You need to drop custom window for skin settings in favour of addon settings dialog? Check pdf http://code.google.com/p/xbmc-gpodder-in...loads/list (page 11) to get list of currently available type of settings.

What for sure is missing now is ability to set addon (already have pull request on that) and [optional] ability to display selected image/multiimage for backdrop selector.

What are benefits? You don't need to maintain yet another window and just use simplified addon settings dialog. You can specify default values (f.e. proper bool defaults without weird inverts, set default backdrops)

And once again - You still we able to use custom window for settings.

Here's some off the top of my head that I can think of -

• A slider control with Min, Max and step values (eg Background Transparency from 80% to 100% in 1% steps)
• A toggle button control (eg Background Selector for Movies, TV Shows, Music, etc)
  

Also can a select box open another select box?

Thanks.