View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006972 | MMW v4 | Burning / Disc Handling | public | 2010-12-12 04:23 | 2022-04-29 01:08 |
| Reporter | rusty | Assigned To | |||
| Priority | urgent | Severity | tweak | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 4.0 | ||||
| Target Version | 4.0 | Fixed in Version | 4.0 | ||
| Summary | 0006972: CD Configuration inacessible from the Rip CD dialog | ||||
| Description | Certain config items e.g. Secure Rip, On the fly encoding that are applicable to CD Ripping aren't configurable via the CD Ripping dialog. Moreover, it's strange that Secure Read would be applied to regular CD reading--shouldn't it only apply to ripping? The issue is all the more relevant because the Accurate Rip dialog allows for the detection of bad rips, but the current workflow doesn't allow the user to change settings on rerip, after such an error has been detected. Possible approach: Move checkboxes for On the fly encoding and secure read to the Rip dialog. To discuss. | ||||
| Tags | todoc-KB | ||||
| Fixed in build | 1344 | ||||
|
|
Yes, the UI is now quite problematic As you indicated [Secure Read] looks like it can be set also for CD playing althouh it is in fact applied only when ripping, so I would suggest: 1. Move Read Type: [Standard Read^] to the Rip dialog, but rename it to: Rip Type: [Standard Read^] 2. eliminate Read frames: [27] Overlap frames: [3] Check frames: [1] [x] On the fly encoding because these 4 controls seems to be little bit old-schooled and used very rarely. 3. eliminate 'CD/DVD/BD Devices' page from options entirely, because now only [x] Read CD-Text is the only remaining control 4. Now, the problem is: Where to move '[x] Read CD-Text' ? Maybe as toggle to the context menu for particular drive, e.g. bellow 'Get Album Info from Freedb', but this doesn't seem to be ideal, any better idea? Eliminating also and leaving this option only in MediaMonkey.ini? |
|
|
1. OK 2. I think that 'On the fly encoding' might be worth keeping for some users. 3. We can probably eliminate Read-CD-Text as an option and always have it enabled. If CD text exists and no other metadata is available, then CD text should be used. If CD text exists, and other metadata is available, then CD Text metadata can be presented in the same dialog as other metadata. See the attached graphic. Summary: the rip CD dialog would have the following options: Rip Type: __________^ [ ] Verify ripped tracks [x] On the fly encoding [ ] Eject CD after ripping [ ] Level Track Volume |
|
|
Reminder sent to: jiri fyi...please review. |
|
|
I think that this kind of simplification would be useful. Just to clarify possible needs for the mentioned (to be removed) options: a. Read frames, Overlap frames, Check frames - this still might be useful for some types of ripping, particularly for older drive models? b. On the fly encoding - again, probably useful for older drives/slower machines when this might prevent ripping problems. c. Read CD-Text - the option to disable is mainly present because it was known problem for some drives that their buggy firmware froze when this option was used. It seems that all a,b&c are useful only for very old, hardly used anymore setups. So, I think it's safe to remove them from UI, but I'd keep them stored in config files, so that they can be modified by user manually. We would just need a short KB article for this. |
|
|
Fixed in build 1336 Note that the old a,b&c options are available in registry: HKCU\Software\MediaMonkey\CDs\ |
|
|
Tested 1336--looks good. Is it possible to have these items configurable via config files (rather than the registry)? It's reasonable to document how to make changes to a config file, but not really how to modify the registry. Also, I think that on-the-fly encoding should be configurable in the UI. Some users will really want to disable that. |
|
|
Fixed in build 1344 1) Added the '[x] On the fly encoding' checkbox 2) Moved CD configuration from registry to MM.INI file, now the drives are stored like: [CD Drive D] ReadSectors=27 ReadType=0 JitterOverlap=3 JitterCheck=1 ReadCDText=1 ConvertOnFly=1 |
|
|
Verified 1348 |
