[vlc-devel] skins2 skin choser

[Tomasz Moń]

Dnia Saturday 22 marzec 2008, Rafaël Carré napisał:
> Skin choser shouldn't be a special config defined in libvlc, just use
> add_string() in the module declaration.

I don't quite know how to do this using add_string()
Please point me the right way.

> Also it spells chooser :)

Renamed.

> You use 2 different ways to define QString path is that wanted or you
> left unwanted debug code ?

One is system global path, the other one is user local.
So it's wanted.

>  QIcon(/*preview.png*/), << are you missing the unzipping code here to
> do what you want ?

Exactly, I'm missing way to gain skin preview.

> +        // Do nothing is skins2 is not the main interface
> +        if( var_Type( p_intf, "skin-to-load" ) == VLC_VAR_STRING )
>
> I'm not really sure how this defines if skins2 is the main interface.

I've based my code on 
modules/gui/skins2/src/skin_main.cpp

I've attached updated patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vlc-skinchooser3.patch
Type: text/x-diff
Size: 10097 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20080322/b2cc5607/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20080322/b2cc5607/attachment.sig>