View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0018292 | MMW 5 | Extensions (bundled) | public | 2021-09-13 19:14 | 2022-10-02 19:36 |
Reporter | drakinite | Assigned To | |||
Priority | urgent | Severity | feature | Reproducibility | unable to reproduce |
Status | closed | Resolution | fixed | ||
Product Version | 5.0.1 | ||||
Target Version | 5.0.3 | Fixed in Version | 5.0.3 | ||
Summary | 0018292: Case Checker: Allow custom strings to force capitalization | ||||
Description | User has requested the ability to add the ability to specify their own strings to force capitalization patterns, such as SuidAkra. As it would be a pain for the user to edit the script themselves, and it is pretty simple to implement, it makes sense to add this feature to the built-in addon. https://www.mediamonkey.com/forum/viewtopic.php?f=32&t=100069 | ||||
Tags | No tags attached. | ||||
Fixed in build | 2616 | ||||
related to | 0016184 | feedback | drakinite | Improve capitalization editing/cleanup |
related to | 0018365 | closed | drakinite | Case Checker, the window does not adhere to theme |
related to | 0018362 | closed | drakinite | Spacing for buttons on Case Checker (regression) |
related to | 0018363 | closed | drakinite | Case Checker, add littleWordString and forceCapString to user customizable setting |
related to | 0018364 | closed | drakinite | Case Checker, present options in an overlay |
related to | 0018366 | closed | drakinite | Case Checker, allow users to determine which tags get updated |
|
Added for 2502. Assigning to Rusty to evaluate the new strings: - "Custom strings to force capitalization:" - "Here you can provide custom strings, like names, to force your own capitalization. Separate them with a pipe | symbol." Note that although the tooltip says "Separate them with a pipe | symbol", I made it so that the user can enter comma-separated or semicolon-separated values and it'll change the separator to a pipe. |
|
1) a) Custom strings to force capitalization: --> Always customize case for these words: Mr|Mrs|OK (I don't think a tooltip is needed with the revised text, as long as commas, |, or ' ' is supported as a separator) b) It's not obvious that clicking 'OK' edits the fields. I would suggest also changing: OK --> Tag c) Future Change: Case Checker --> Edit capitalization Suggested changes: 2) The script already edits the case of articles appearing mid-string. Would it make sense for this to be configurable so that it's useful for other languages? e.g.: Customize case for these words in mid-string: a an and as at but by feat for from in into mid nor of off on onto or over per the to via with 3) Would it be possible to have checkboxes next to each track to allow users to accept/reject changes (working similarly to Auto-tag)? If yes, add an additional string and change the font size (like in Auto-tag): Recommended changes to capitalization --> Recommended changes to capitalization. Choose which files to update: |
|
Raised priority so that at least the new strings are updated for 5.0.2. |
|
Strings updated in build 2512. Deferring the other features to 5.0.3. |
|
Added two strings for 0018363: Lowercase common short words, Capitalize common abbreviations. Also, perhaps "Specify custom case for these words" is better than "Always customize case for these words"? |
|
How about: How about 'Customize case for these words' -- Re. offline comment: about using 'OK' instead of Tag: 'OK' would be fine if the dialog would communicate that it's editing track properties e.g. Recommended changes to capitalization --> Recommended changes to capitalization Update properties ? (note: this is an existing string) OR Recommended changes to capitalization Highlighted fields will be updated (note: this is a new string) |
|
Strings updated in 2514. Header is now: "Recommended changes to capitalization - Update properties?" [two separate strings: "Recommended changes to capitalization" and "Update properties"] and the custom case string is now "Customize case for these words" |
|
Overhauled Case Checker's appearance & made it display as a list view with checkboxes. Took heavy inspiration from Auto-Tag to define its behavior, in order to provide a more cohesive experience for the user. Header is now "Recommended changes to capitalization - Choose which files to update:" |
|
Verified functionality in 2611 Left resolved for language and header text confirmation. |
|
Reopen Check boxes are not aligned. If possible Check Box column should not be resizable. |
|
|
|
Re checkbox alignment: Updated Metro M to fix it. Re checkbox resizing: As it's not currently possible to specify individual columns as non resizable, this is out of the scope for 5.0.3. Maybe it can be added as a feature for 5.1+. |
|
Verified 2616 Alignment fixed. Re checkbox resizing: Ok, if that was not trivial we will leave as is. We can open new bug for 5.1 if needed. |