| User review:
|
@sarkas:
OK, so there's a few considerations to be added:
- It would make things easier to understand, if you didn't intermix "Settings" and "Options", or clearly said that both mean the same to you, and are used without distinction in your app.
- It would make things easier to handle, if you mentioned that ALL options need to be specified in the Options panel for the app to exercise anything useful (what's the use of the "enable" checkbox in that case, anyway?).
- Your app shouldn't bomb out with "Add dirs in Settings first", if e.g. one wishes to one-click-convert a ts to mkv, and hasn't enabled mp4box in Options panel. mp4box isn't used at all in that scenario, but your code in Main.cs explicitly checks for this setting to be configured and a filename provided. (I didn't want to d/l or install mp4box, so simply renamed mkvmerge.exe to mp4box.exe, which circumvented that programming flaw.)
- One doesn't have to add "dirs" on the options panel (there's no way to do that, anyway), one can only add file specifications (location incl. filename)
- There simply is no option or setting to set an output directory for a one-click conversion in this app. Neither in the github readme. The output file is created in the same folder as the input file, using the same basename as the source. I didn't check what your app would do in case the input location was read-only...
- I didn't take time to imagine or to find out what the intention could be for the parameters "DefaultOutputDir" and "LastUsedOneClickFolder" in settings.xml.
But thanks for pointing me at your project folder on github, and providing the source there. That helped me in understanding the app's behavior.
Perhaps all this should better get discussed in a thread in the forum, rather than posting this as another review...
Finally, please accept my excuses. I didn't mean to offend you in any way. I was just disappointed (hence the alias), because the app didn't work as I had eagerly hoped for, and didn't provide ease of use for my personal needs. |
