[PATCH] WinGUI Patches
Forum rules
*******************************
Please be aware we are now using GitHub for issue tracking and feature requests.
- This section of the forum is now closed to new topics.
*******************************
*******************************
Please be aware we are now using GitHub for issue tracking and feature requests.
- This section of the forum is now closed to new topics.
*******************************
Re: WinGui: Small icon for Queue window Re-Add Current Job
You'd save yourself some time if you just created a single post with a list of the smaller things you spot which needs sorted.
I'll take a look at it in the morn.
I'll take a look at it in the morn.
Re: WinGui: Small icon for Queue window Re-Add Current Job
Absolutely, I will bundle the small stuff in future.
I tried to bundle the related matters, and this probably should have gone in my other thread. Some development communities prefer to keep separate issues, no matter how small, completely separate, and others want to keep the proliferation of threads, silos, communications, etc., to a minimum. I'm still learning how this community operates.
I tried to bundle the related matters, and this probably should have gone in my other thread. Some development communities prefer to keep separate issues, no matter how small, completely separate, and others want to keep the proliferation of threads, silos, communications, etc., to a minimum. I'm still learning how this community operates.
Re: [PATCH] WinGui: Prompt to confirm GUI closing
http://trac.handbrake.fr/changeset/2260
I figure if it ends up bothering me, I'll add it, otherwise I don't feel it's worth the effort.Is this something others would want, or would find annoying? Is it worth adding a config option to enable/disable it?
Re: [PATCH] WinGUI Patches
All patches merged into a single thread. Please keep patches to this thread, it'll save cluttering up the dev forum.
Re: [PATCH] WinGUI Patches
http://handbrake.fr/pastebin/pastebin.php?show=508
The default psy-rd value is 1.0. This patch sets the GUI slider to the default (10 for the slider) so it matches the text display when no psy-rd value is specified.
The default psy-rd value is 1.0. This patch sets the GUI slider to the default (10 for the slider) so it matches the text display when no psy-rd value is specified.
Re: [PATCH] WinGUI Patches
Good catch! Thanks
http://trac.handbrake.fr/changeset/2290
http://trac.handbrake.fr/changeset/2290
Re: [PATCH] WinGUI Patches
Should line 658 in QueryParser.cs be changed from
to
so it loads the videoFramerate correctly when it is set to a framerate with a decimal place by the preset.
I have noticed that if the preset is set to 29.97 it will default to "Same as Source" or 29 instead of "29.97"
Code: Select all
Match videoFramerate = Regex.Match(input, @"-r ([0-9]*)");
Code: Select all
Match videoFramerate = Regex.Match(input, @"-r ([0-9].*)");
I have noticed that if the preset is set to 29.97 it will default to "Same as Source" or 29 instead of "29.97"
Re: [PATCH] WinGUI Patches
Yup, it should be. Thanks.
Re: [PATCH] WinGUI Patches
http://handbrake.fr/pastebin/pastebin.php?show=529
- A few fixes for destination filenames:
- Separated the filename from the initial directory before showing the save dialog. This way, even if there is bad or no directory info, the filename is preserved.
- Changed the check for the destination path. With a previous fix, the user could still get an exception if they removed the file extension.
- Added manual file extension check. FileDialog.AddExtension doesn't work well with filenames containing dots.
- Changed DVD_Save.CheckFileExists to .OverwritePrompt. CheckFileExists only lets you save if a file does already exist. I've made an assumption that OverwritePrompt is what is desired, where the user is prompted to overwrite an existing file.
- Created a new public function frmMain.setContainerOpts() to enable/disable the MP4 container options whenever the file format, video encoder, or preset is changed. This is based on the file format selection, not the destination filename. If the user picked a different extension or no extension, it was possible to get into a scenario where the container options wouldn't enable when MP4 or M4V was selected.
Re: [PATCH] WinGUI Patches
Checked in.