Xrecord poster dimensions are skewed

[Wakalaka]

noposter.gif is 100x150, which looks right in gb-pvr. But my show icons are 320x240, and they get skewed to 100x150 which makes them look weird. Is there a way to force noposter.gif to 320x240 so the dimensions are all the same? Or some ratio of 320x240, e.g. 160x120, if it's an issue of fitting so many icons in a fixed amount of space?

[fla]

These things are skinnable in gbpvr\skin\Blue\xrecord\skin.xml.

Play with things like
<Image height="110" width="84" textbox="60"/>
<Advanced active="true" rows="2" columns="4" box="true" startGap="80" rowGap="110"/>

Xrecord is supposed to be able to download posters from imdb.com and tv.com. Since a lot of the posters on these web sites have aspect ratios like a dvd box (tall and narrow) the default skin uses that shape.

[Wakalaka]

I have a bunch of ideas to make Xrecord a more error proof and cleaner plugin. Do you mind adding another developer to the effort? I don't know of alternatives, and I've never used GB-PVR's built-in support for tv listings, recording, scheduling, etc. I'm sure Xrecord is the best, so I'd like to make it better.

I'm modifying skin\Basic\xrecord\skin.xml and almost have it looking the way I want it. I keep changing the file, reloading GB-PVR, and checking the icon placement. I can't determine which parameter controls the x position of the first column. I want to move it to the left, closer to the buttons so there isn't as big of a gap between the buttons and the first column of show icons.

[fla]

This thread talks about getting the source code from sourceforge. I suggest you get the code and do your stuff. If the owners Jorm and Kingargyle think along the same lines, they will add you as a sourceforge developer. They are happy to have their work continued by others, that's why the project was put on sourceforge.

A quick look at the code and I see some hardcoded values horizontally but skinnable vertically.

Code:

int verticalOffset = startGap + ( rowGap * column+1);
...
int horizontalOffset = selectNewsStartX + 110 + ( 130 * row+1);

If you add skinnable items, make sure to add them to both the Blue and Red Sky skins.

[Fatman_do]

There are also some posts where there are users requesting more flexibility in skinning the amount of rows and columns. Also breaking the 720x480 hardcoding so this plugin can be added to higher definition skins. I would by happy to help out with any skinning issues or questions.

I'll take a look at the source code just to see what it looks like. I can't code really in any programing language, but I can "see" things. I just don't know how to fix them.

[fhmanas]

Easy enough to have more than the default 8 icons by manipulating the skin but I saw in the code that it is hardcoded to only allow 8 items at a time, so even if you have 12 or 15 items on screen, when you reach beyond the 8th item, it will execute a next page. If you are willing to fix it, please correct this as well.

[jksmurf]

I have a bunch of ideas to make Xrecord a more error proof and cleaner plugin.

I'm not a developer but I really really like XRecord (esp. its simplicity) and use it almost exclusively on the MVP. Glad to see others think the same.

k.

[KingArgyle]

As Fla has said, download the source, make your changes, and for now, I would say post them back to here, or provide a patch for the changes on the CDK and Xrecord suite bug tracking site.

Bugs, Feature requests, and patches can be added at the following site:

https://sourceforge.net/tracker/?group_i...tid=772447

The best way to make sure that an enhancement or bug is fixed is to also add it to the bug tracker. If you do make a patch, please post that as well, so that Ton, Fla, or myself can review it. Ton and Fla are more active in development, but if they are happy with it, we can get the patch added fairly quickly.

I'm always willing to give some access, but like to see some code contributions back before doing so.

[fla]

Easy enough to have more than the default 8 icons by manipulating the skin but I saw in the code that it is hardcoded to only allow 8 items at a time, so even if you have 12 or 15 items on screen, when you reach beyond the 8th item, it will execute a next page. If you are willing to fix it, please correct this as well.

This thread discusses that bug which was closed almost a year ago and I just tried it now with the current xrecord release and can't reproduce the problem. If it still happens in the current xrecord release please add a comment to the above thread or to the bug below on sourceforge.

[ 1303881 ] Bug: Navigation problems if you columns =3

[fla]

I have a bunch of ideas to make Xrecord a more error proof and cleaner plugin. Do you mind adding another developer to the effort?

I just submitted a pretty significant number of skinning related fixes to XRecord (only xrecord not xsearch and xguide) on sourceforge. If you plan to make any changes to xrecord, you should do a CVS update first if you've already checked out the code.

[ 1677526 ] XRecord basic skinning doesn't work