Questions [Cover / Popup / Variables]

[Maverick07x]

Hey guys,
im the developer of the FusionX and MetroX skin on deviantart, and i have some questions to improve the skins.
http://maverick07x.deviantart.com/art/V ... ck07x&qo=1
http://maverick07x.deviantart.com/art/V ... ck07x&qo=0

So first of all the cover art display.
If i active the "Replace image with cover art" button at a random image, the player will crashes.
Well it starts but in the same second vlc closes without any error message.
Any idea?


Next one...
It does look like the popup menus does not work.
so fe. if i set the action for a random button to: show dialog box => audio settings popup menu
nothing happens?
=> well looks like i found the solution^^


last one...
is there a way to use random variables instead of default boolean ones?
the idea is to switch the color of the skin by pressing a button without changing the layout.
Cause if you do it the normal way - the skin size will change to the size of the layout.

So fe. the skin has now cyan bars - if i press the button to get magenta ones, the cyan will be deactivated and magenta activated.
ive made this trick at the MetroX skin to show if the EQ is on or off.


thats it for now =)
- maverick07x

[Jean-Baptiste Kempf]

Hey guys,
im the developer of the FusionX and MetroX skin on deviantart, and i have some questions to improve the skins.
http://maverick07x.deviantart.com/art/V ... ck07x&qo=1
http://maverick07x.deviantart.com/art/V ... ck07x&qo=0

Wow. Just wow.

So first of all the cover art display.
If i active the "Replace image with cover art" button at a random image, the player will crashes.
Well it starts but in the same second vlc closes without any error message.
Any idea?

Share a skin that makes it crash, I will fix it.

Next one...
It does look like the popup menus does not work.
so fe. if i set the action for a random button to: show dialog box => audio settings popup menu
nothing happens?
=> well looks like i found the solution^^

?

last one...
is there a way to use random variables instead of default boolean ones?

Don't think so, but we need erwan for that.

[Maverick07x]

http://www.share-online.biz/dl/H8I3E9HMH4
heres the skin with the cover crash
- sorry for the bad provider =/

and ye the popup menu is working now - looks like i had to update vlc ^^

[erwan10]

Hi and thanks for reporting these skins2 issues.

A few things :

- Art display has been misfunctioning in latest vlc (nightly builds). This should be remedied with nightly builds starting from tomorrow on.

- As a test case, I downloaded the MetroX_Cyan skin from your original website and altered it in a way I know it should work. The result can be found at http://www.mirari.fr/QUkJ . It works very fine for me on Linux with latest vlc from today. Feedbacks welcomed if you do your own tests.

- I failed to download the skin at http://www.share-online.biz/dl/H8I3E9HMH4 .... too many warnings with my virus scan. Please, select another more friendly place (this one seems friendly : http://www.mirari.fr)

Regarding using random variables instead of default boolean ones, it is not possible, and as the skins2 engine is designed, it will be of little value. You're right about this layout issue where resizing only takes place on one layout, not all of them. Yet, that is not easy to fix with the current design ....

[Jean-Baptiste Kempf]

Thanks erwan!

[Maverick07x]

Nope it does not work. =(

When i open the skin with the skin editor i get an error:
org.xml.sax.SAXParseException_ Element type Image must be followed by either attribute specifications > or />

thx, i uploaded it here again
http://www.mirari.fr/pCO6

[erwan10]

Ok, the skin you provided does have a problem.

The problem comes from the VLC skins editor that failed to format the theme.xml well. One space must be added just before art="true" (see the faulty line from theme.xml here below) .... As it stands right now, the theme.xml file is not xml-compliant.Once this extra space was added, you skin works very well. It just needs a small change if you wish to preserve the art aspect ratio. here is the same skin with the space corrected : http://www.mirari.fr/aSz3

Code: Select all

<Image id="mainframe_cover" image="main_cover"art="true" resize="scale" rightbottom="rightbottom"/>

Maybe, please 3breadt for some help to fix this Java issue.

I guess the problem you meet when editing my skin is also related to the VLC skins Editor with regard to the art parameter.

Did you try to test the skin just by itself (without the skins Editor) ? Did you also have problems with that ?

[Jean-Baptiste Kempf]

The problem comes from the VLC skins editor that failed to format the theme.xml well.

Can't you fix it?

[erwan10]

The problem comes from the VLC skins editor that failed to format the theme.xml well.

Can't you fix it?

Do I have access to the source code ? just email me how to access it and hopefully I'll fix the issue.

[Maverick07x]

great, it works now.
thanks

one more question...
is there a way to put and force an object/image into the right bottom corner of the display?

The idea is to display the cover art and some infos at the right bottom corner - activating it via button
the problem i will get is that i have to specify a position (fe. 0,0 => top left corner)

[erwan10]

Yes, the skins2 engine supports geographical placement. You can create a window and tell this window to be placed at the SouthEast of the display (see the position, x-margin, y-margin, x-offset, y-offset parameters at http://www.videolan.org/vlc/skins2-create.html )

But, .... I don't think the skins2 editor has implemented it yet. You may need to do it the old way by editing the theme.xml file with any text editor and add these parameters manually.

[Maverick07x]

great thanks - worked perfect, but all work for nothing...
thought that i can enable the AlwaysOnTop function for a single window - sadly its not possible - so the coverart will be in the videoframe =(

[Maverick07x]

ive updated both skins - added coverart and some fixes...anyway

so there is no way, even in the future, that you add custom variables for skins?

[erwan10]

I just tested them.. I like them

Just one thing, don't forget to check the logs since it helps debug a skin. Yours have got several errors. There are a few occurrences of "Mainwindow" that should obviously be "MainWindow" (It is case sensitve), and a missinfg main_coverart.png file.

Code: Select all

[0x8a05560] skins2 interface error: missing file: main_coverart.png [0x8a05560] skins2 interface error: unknown window (Mainwindow) [0x8a05560] skins2 interface error: unknown window (Mainwindow) [0x8a05560] skins2 interface error: unknown window (Mainwindow) [

For custom variables, why not, but it needs some other work to be done beforehand to be truly usable.

[Grumpus Maximus]

I understand this thread is over a year old but I am having the same issue with cover art. Your response to Maverick07x makes sense and I have done this by editing the .xml file in a text editor (Notepad). If I then open the .xml in the Skins Editor and try to test in VLC, it does not work. When I close the Skin Editor and then edit the .xml again in Notepad, VLC will load correctly and show my skin. But when I export my skin to VLT, the skin does not load. I can open the .xml in VLC after making changes and it loads correctly. So how are you able to make this change and keep it when exporting to VLT. I have opened the VLT file in 7-Zip and edited the .xml there but it does not work. I am using a very simple line:

<Image id="Black_NowPlaying_AlbumArt" x="31" y="110" image="Background_AlbumArt" art="true" resize="scale"/>

As I said, if I make this change in Notepad and open the xml in VLC, the skin loads fine and cover art is displayed. Only when I export to VLT it will not work.

Kind Regards, Grumps