bordertexture issue

[Jeroen]

I updated my Windows build to rev14582 today and I noticed a problem in my skin.

A commonly used item in my skin is:



<itemlayout height="90" width="380">
<control type="image">
<posx>7</posx>
<posy>7</posy>
<width>370</width>
<height>68</height>
<aspectratio align="left">keep</aspectratio>
<info>ListItem.Thumb</info>
<bordertexture border="2">-</bordertexture>
<bordersize>6</bordersize>
</control>
</itemlayout>

<focusedlayout height="90" width="380">
<control type="image">
<posx>7</posx>
<posy>7</posy>
<width>370</width>
<height>68</height>
<aspectratio align="left">keep</aspectratio>
<info>ListItem.Thumb</info>
<bordertexture border="2">list-focus.png</bordertexture>
<bordersize>6</bordersize>
</control>
</focusedlayout>


In the case of the unfocused layout no bordertexture is needed, so I leave that open. It would however always add the border, just no texture.
The border doesn't get applied anymore though when unfocused, resulting in different sizes, making the selection....jumpy....
I'm 100% sure it wasn't this way yesterday and I haven't touched this particular code since.

Intended changes? Bug?

[skunkm0nkee]

I'm not sure what has changed in XBMC to break your code but you could get around by adding width, height and posx, posy tags (itemlayout would be slightly smaller and positioned to match up to focusedlayout)

[jmarshall]

Probably "unintended that it worked in the first place" ;)

Will take a look.

Cheers,
Jonathan

[Jeroen]

I'm not sure what has changed in XBMC to break your code but you could get around by adding width, height and posx, posy tags (itemlayout would be slightly smaller and positioned to match up to focusedlayout)

yeah, those are all present, the above was just a simplified example.

changed it to a complete c+p :)

Probably "unintended that it worked in the first place" ;)

I was afraid of that :laugh:

[jmarshall]

Should be fixed as of rev 14586.

Cheers,
Jonathan

[Jeroen]

confirmed, many thanks :)