BurnDVDx2

[dark_half]

Just wanted to let someone know that there is a font problem with unselected text being incredible small. In the skin.xml I changed the following on 2 lines to correct.

Code:

textStyle="ListSelectedIn[color=Red]c[/color]active"

to

Code:

textStyle="ListSelectedInactive"

[Fatman_do]

Thanks

[roy]

I also noticed that the font used for the @nowPlaying text in Music Library skin didn't look right because it used GeneralTextStyleItalic, which is not defined, I think. I fixed it so now I can't double check what it was.

[Hairy]

I also noticed that the font used for the @nowPlaying text in Music Library skin didn't look right because it used GeneralTextStyleItalic, which is not defined, I think. I fixed it so now I can't double check what it was.

"GeneralTextStyleItalic" was an intentional inclusion, it was added for aesthetical reasons and is defined in the baseskin.

This is not a justification for it but an explanation of why I used it.

After sub reintroduced panels I struggled to think of exactly where to position them in the reflect theme (naively it wasn't designed with them in mind). I decided that for now top right seemed to make sense and as a result I felt I had to rearrange the layout of the default CS3 skin (Reflect) to accommodate them. So thats why I rearranged the main menu and some of the plugin layouts. When I did I felt that the "@nowPlaying" text needed to stand out or be defined somehow, so I added the "GeneralTextStyleItalic" textstyle (it's also used for some sub-title formatting in the search plugin) to differentiate it.

I'm rapidly learning that most of the formatting is the way it is for very good reason and that even small changes to style, position and especially font size has a much larger knock on effect (compatibility wise) than it ever appears when I doing it. This makes it very difficult to know what you can or should change.

When I get a little more time I'm gonna recheck what I changed for this release and see what didn't work out so well. Oh and hit up Fatman_do for advise on some of the basic no - no's

Cheers

[roy]

"GeneralTextStyleItalic" was an intentional inclusion, it was added for aesthetical reasons and is defined in the baseskin.
Cheers

I guess that's why it didn't work for me, as it wasn't defined in my custom baseskin.

Keep up the good work with Reflect.