BurnDVD plugin (a preview, supports PAL only)

[pastro]

oops... it was something wrong with my player...:o
and this is with the beta you gave me..

i did see a couple of problems with quotes and paths and such..
had to edit my batches to fix cuz my temp is on another drive in long filename dir..
oh and found a filename problem, i have a filename with a '&' in it..
even the short name! so it still needs quotes if short name used..
["G:\mpg\1ST&TE~1\1ST&TE~1.MPG" works, not G:\mpg\1ST&TE~1\1ST&TE~1.MPG]
so best just to use all long filenames and always quotes, won't have any probs then..

and the ffmpeg add didn't seem to work because the current path wasn't the gb dir, but the dvd temp dir...so can't use ..\ffmpeg\ffmpeg..
i hardcoded them and works for me..
other than that, works great...good job!

wishlist ideas:
for menu structure, have your list style for first page, then when clicking on them go to that movie's menu page with full descriptions like mine, and click there to play...or back to main menu..
that way have index at a glance, but with room for more info on individual pages...and play buttons on list also..
i know it'd be tricky to work out, but would be nice...

or just put descriptions on main page, bigger boxes...

and maybe use ff libs to detect incoming movies size and adjust the -v for the right width...[4:3/16:9 and 352xXXX or 720xXXX] dvdauthor defaults to ntsc,4:3,720x so don't need any -v for those.and it autodetects what it can..
so i used no -v so it would autodetect 352x and 720x movies..but it stays same for all movies on dvd so detection is best...
so that it can take any size files and still look right..and maybe while at it, detect dvd compat. and warn if not in spec...
:eek:

I'll check out the filename problem, and the path to ffmpeg. Doesn't your buildDVD.cmd have a:
cd \program files\devnz\gbpvr\third party\burndvd in it? That was why the path to ffmpeg was ../ffmpeg/ffmpeg.
I think I'm going to make it look more like showburner for the menus. Most of that is doable from the skin, but the menu image size will need to be changed in the program.
I had four lines for the description, I thought that was enough. Apparently not. When I change over the menu, I'll fix that too. I'm going to add the episode if it exists as well.
The next beta has a button to toggle for 16:9 and 4:3. Autodetect would be good, but for now the button should handle it.

[pastro]

btw, thanks for figuring out the new config.xml + separate add method for dvdauthor..
now i can use a single newer version in mine...

That was tmrt's doing. Very slick.

[pBS]

oh yea...i liked the way he used the fifo's too..

yea, my cmd does have a cd command in it..sorry..i was running it manually..
but the filenames prob does exist...easily solved tho..

you may want to size the boxes as big as you can get them [use more screen space] and shrink the font for the descriptions only..
i'll see if i can come up with a way to have a single box for descriptions that changes as you highlight diff shows...i know it can be done, just not sure how hard to implement...

and your buttons seem ok...i think it actually means literally 'odd' numbered pos. [51 instead of 50]
something about some dvd players and lower resolutions looking funny, but shouldn't ever make it not work...so those are fine as is..
looking good

[pastro]

oh yea...i liked the way he used the fifo's too..

yea, my cmd does have a cd command in it..sorry..i was running it manually..
but the filenames prob does exist...easily solved tho..

you may want to size the boxes as big as you can get them [use more screen space] and shrink the font for the descriptions only..
i'll see if i can come up with a way to have a single box for descriptions that changes as you highlight diff shows...i know it can be done, just not sure how hard to implement...

and your buttons seem ok...i think it actually means literally 'odd' numbered pos. [51 instead of 50]
something about some dvd players and lower resolutions looking funny, but shouldn't ever make it not work...so those are fine as is..
looking good

I fixed the filename problem, thanks for finding that.
There was an offset calculation in the burndvd plugin that caused the odd number for the button boundary. I made this an even number and the warning went away.
I think I'm going to go more with your layout that way I can add more text easily and get the bmp to look right. I'll leave 8 lines or so for the description. That should be plenty since there will be more room on the line with the bmp above it.
There is one last problem when you select files with the mouse sometimes causing an error. Selecting with a return causes no issue.

[pBS]

oooo,if you'll have a thumbnail image, can it be user selectable?
i had lots of requests on that one...
i know a player would be hard to implement for selection, so maybe just a slider, that when every time it's let go,would make snap and show it for acceptance..
no motion needed really...and maybe a seconds +- arrows for fine tuning..
not sure if there's a slider object available tho...

[tmrt]

so best just to use all long filenames and always quotes, won't have any probs then..

I suggest to stick with short filenames and quotes because some nonenglish unicode characters are not translated properly in batch files.

[tmrt]

There was an offset calculation in the burndvd plugin that caused the odd number for the button boundary. I made this an even number and the warning went away.

I always got warning, but I could not figure what is the problem. Does the button selection functions now? In my case button selection works perfectly on my two Philips DVD players and in software player despite the warning.

[pastro]

I always got warning, but I could not figure what is the problem. Does the button selection functions now? In my case button selection works perfectly on my two Philips DVD players and in software player despite the warning.

I think it works fine. Some software players were having trouble but it didn't make any difference changing the button offset.

[pastro]

I suggest to stick with short filenames and quotes because some nonenglish unicode characters are not translated properly in batch files.

That's what I did.
In burndvd the following line had the trouble

int hr = GetShortPathName(fileObject.Longname, sb, sb.Capacity);

I added this after the line:
sb.Insert(0, '"');
sb.Append('"');

[pastro]

oooo,if you'll have a thumbnail image, can it be user selectable?
i had lots of requests on that one...
i know a player would be hard to implement for selection, so maybe just a slider, that when every time it's let go,would make snap and show it for acceptance..
no motion needed really...and maybe a seconds +- arrows for fine tuning..
not sure if there's a slider object available tho...

It's there already. Look in the edit tool.