View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002062 | MMW v4 | Burning / Disc Handling | public | 2005-10-09 03:21 | 2006-02-16 17:05 |
| Reporter | rusty | Assigned To | |||
| Priority | urgent | Severity | feature | Reproducibility | always |
| Status | closed | Resolution | duplicate | ||
| Summary | 0002062: Data CD Masks implementation is incomplete | ||||
| Description | The new mask configuration is only half implemented. We should either revert to the format consistent with other dialogs (i.e. change the order of the masks and remove <\> and <-> buttons from Burn wizard) OR preferably, try to implement the spec fully. The remaining items to implement the spec fully are: a) Group the masks as specced in 0000349 <Title> <Filename> <Track#> <Play Order> <Random> ----------- \ (directory) - (splitter) ----------- <Artist> <Album Artist> <Album> <Genre> <Playlist> <Year> <Composer> <Bitrate> <BPM> <Path> <Custom1> <Custom2> <Custom3> b) Have all the masks appear via a dropdown button to the right of the mask c) Add comments for some of the mask items (e.g. '\' and '-') This is not high priority so if it's a lot of work, we should postpone this and revert to the original format. I'd like to see this though, and give it a try to determine whether it could help make masks easier to understand. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 919 | ||||
|
|
Re. a) I'm not sure if undersand correctly. I had changed the order of masks as specced in 0000349 before Alpha 3 release. So Do you mean only add <Playlist> which is missing and convert: <Auto Number> -> <Play Order> <\> -> \ (directory) <-> -> - (splitter) Do I understand well?? Btw. The mask labeled '\ (directory)' is too large instead of <\>. Are you sure that this is a good solution? |
|
|
Ludek, What I meant is that: b) - The button should be to the right of the dialog - The masks should appear vertically dropping down from the button a) The masks that drop down should be divided into groups via splitters c) - It should be possible to have explanatory text next to the mask (e.g. '(directory)' is an explanation of the '\' entry. Note '\' and not '<\>' Note: I'm not certain that this will be much easier, however, I would like to give it a try if it's not to difficult to implement. As far as the <Playlist> mask, don't bother with it as part of this bug--it's covered under bug 0002068. |
|
|
Raising to 'immediate' since we need to either decide to fix it this week or push it from 2.5 since we want to finalize strings this week (or add the strings to the current build so that we can decide to add it in later on in the release cycle). |
|
|
I'm also not sure if it is easy to implement, but for now, I have added the strings 'splitter', 'directory', 'Play Order' to be localized which could be useful in the future. Take it into account and re-assign to me, please. |
|
|
I just found that it is very simple to implement, I will do it shortly. |
|
|
Tested in 919 and this is looking really good. There's one usability problem though: the user must repeatedly click the '>>' button in order to see the masks. Could you modify the implementation so that whenever the cursor is active within a mask, the list of masks is shown? (i.e. the '>>' button should be pressed and stay pressed automatically as long as the cursor is in the mask combo box). This way the user can see all available masks, grouped, with explanations, whenever they are needed. |
|
|
Per discussion with Jiri it isn't necessary. I think so too. |
|
|
Per discussion: this can be pushed as it requires some experimentation and we don't have time right now. |
|
|
Note: mask implementations across the product should be rationalized. |
|
|
Resolving--superceded by 2370. |
