View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006665 | MMW v4 | Framework: Skinning | public | 2010-11-09 06:24 | 2010-11-14 04:26 |
| 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 | 0006665: '-' Appears in Player | ||||
| Description | I'm not sure if this is a bug, a skinning problem, or a problem with the masks describing the format of the currently playing track (I suspect it's a combination of these issues, since the bug appears differently depending on what skin is used--in fact I wonder if it's a bug that skins allow this to be specified, since it will usually conflict with Summary formats as described in 0006571). For a track in which metadata is missing (e.g. Title exists, but no Album or Artist): Using the Pulse skin, the following appears: <Title> - - Using the Vitreous skin, the following appears: Unknown - <Title> In both cases, I would expect the following to appear: <Title> According to Frank, the following settings are used for the Pulse skin: (when no track is select the 2 "-"from mask showing up! [SongTitle] Type=Text Transperent=1 Resizable=1,0,1,0 Pos=129,19 Size=350,18 Transperent=1 TextMask=%S - %A - %L FontColor=424242 FontHeight=-15 FontName=Calibri TextAlign=-1 | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1324 | ||||
|
|
I think this is a matter of opinion. As my opinion is that the current behavior is appropriate (with perhaps the exception of " - - " appearing when nothing is selected in Pulse) as if it should display "Artist - Title - Album" and there is a title missing and the " - " is omitted, then you end up with "Artist - Album" which could be misleading or result in some confusion as to which fields are being displayed if only two are present when normally there are three for example. |
|
|
I think we don't need to do anything on this part as Frank can use $if command in mask. Like : %S$if(%A, - ,)%A$if(%L, - ,)%L |
|
|
To clarify, the default summary mask should be along the lines that Petr describes. As far as the skin goes, the Summary object should be used instead of the old SongTitle object (which is being retained for compatibility with older skins). i.e. in the skin's ini file, rename SongTitle to Summary and remove TextMask. |
|
|
verified 1325 |
