Anime Library - Support Thread

[Ommina]

seriously, dont worry, I wouldnt report the bugs if I didnt think you could fix them
I too am a developer, just for much lower level stuff (device drivers), and I completely understand that programs have bugs, and they can be fixed. and usually when someone says "OMG IT DONT WORKS" its pretty hard to tell exactly what was going on, so I will apologize for not giving an adequate description of the problem I was experiencing (it was late, I wanted to sleep)

More importantly, thank you for taking the time to report it.

I fear the default response when coming up against a bug in any sort of free software (be in a plugin, gbpvr itself, or generally) is to shrug one's shoulders, say "oh well", delete it and move on. And it's not a wholly unreasonable response, either; not everybody has the patience or technical know-how to work though the issues. They just want it to work.

Still, I'm personally bothered knowing that my own work is misbehaving for somebody else. And I'd like to get it fixed given the opportunity.

So I appreciate the post.

--

As for the skinning, you should be able to do all that without too much fuss. I regularly make changes to colours, backgrounds, styles and the like through the CommunitySkin plugin thingie.

That said, I have not looked at it at all yet - the screen not rendering correctly has taken my attention so far. But I still plan to give it a try tonight.

[Ommina]

OK, let's try this one.

It's a bit of a hack, but I'm hoping, if nothing else, it will provide some guidance as to what is going wrong.

[Ted the Penguin]

Whatever you did (I am assuming the more unique names?) that worked, everything seemed to play nicely.
Thanks for the fix!

Is it just me or is your logging really noisy (pvrx2.exe.log, using the normal app)?

[Ted the Penguin]

hmmm.... now this one is interesting. youre creating (copying?) button images into your skin folder button_normal and button_selected

[Ommina]

Whatever you did (I am assuming the more unique names?) that worked, everything seemed to play nicely.
Thanks for the fix!

Is it just me or is your logging really noisy (pvrx2.exe.log, using the normal app)?

Yep! I intercepted the names supplied and changed them into GUIDs. You want unique? I'll give you UNIQUE!

OK, not an ideal solution at any rate, but on the other hand, if is an effective solution, that counts for something. Ultimately, I think this one will fall to JavaWiz for a "proper" fix, since I use his UI Library for the UI. Since the rendering problem will be affecting his own plugins as well, I expect he'll want to hear about it. (The vast majority of that logging your seeing comes from his library as well - I believe it has been reduced since then. I'll ask about it.)

And with that behaving better, I'm going to try the Sassari skin now. Good thing today was a holiday!

[Ted the Penguin]

ok... figured out what is going on with some parts of the skinning...

youre not actually drawing the list background behind the list (on the screen) but as part of the list?

youre not using the standard format for the listview placement
yours:

Code:

<Placement name="ListView" loc="23.2,4" size="54.1,4.5" [B]offset[/B]="5.0" rows="18"/>

video library:

Code:

<Placement name="ListView" loc="23.2,4" size="54.1,4.5" [B]spacing[/B]="5.0" rows="18"/>

why aren't you pulling the standard menu button (the ones on the left) placement?

[Ommina]

Sadly, I think we're coming up against what happens when I figure out skinning by grabbing a couple of examples and poking things until the looked good. It works, and I learn just enough to be dangerous, but the results are a bit hit and miss.

Still, while mowing the lawn, I decided the screen rendering fix (JavaWiz has already provided a new version of his library to provide unique names) really does justify a another full release of the 1.0 version of the library. I'll merge in the fix for the three day pause as well.

So, with that in mind, I'll start by looking at the standard formats you've listed above, and see if I can coax mine to do the same. Then I'll release a new set with (hopefully) a better behaved skinning experience.

[Ted the Penguin]

heh... I really have no idea how things work behind the scenes with the skin stuff, but some plugins (weather, cinema) use the same exact names as the built in plugins for the skins. it would be really nice if all plugins used that same framework

just let me know when you want me to test out your release candidate, to make sure that the library fix takes care of the problem I was seeing.

I am going to continue hacking and slashing at the skin, I got it looking closer to the rest of sassari, but still not 100%. (I swear I will release that damn skin someday!)

[Ted the Penguin]

ohh, just thought of another skinning thing... you dont use the "screen name", to display the name of your plugin.

[Ommina]

Oh, I'm a big fan of using the same conventions as sub's skins if at all possible, so I'm quite anxious to bring things in line with the style he uses as best I'm able.

Spent some time on it last night on the belief that changing to the StandardButtonOrigin as defined in the baseskin would be easy and painless. And, of course, ran straight into an error. (Of course). So, time to get another clarification from sub before I can go any further in that direction.

As for the Screen Name, yeah, that's something I added in for the 1.7 release. I confess I added it only because it seemed like the thing to do, and to conform to the other plugins. I personally could never see the point, it always struck me that it was pretty apparent what plugin I was using, why the necessity for the big label?

But I bit the bullet and added it. (And promptly removed it for my copy!)