Bugzilla – Bug 17081
change VA category setting to not be blank, ie. use/show the default and improve text
Last modified: 2015-10-15 03:57:38 UTC
in the SBS settings, there is this option: "When compilation albums are grouped together, they appear under "Various Artists" by default. You can change that name below." the option works but the problem is the field is blank by default even though "Various Artists" is whats currently being used by default. 1. so the above text should read as: "When compilation albums are grouped together, they are shown under the name in the box below. This box can't be voided, but you can change the name below." 2. and that box should have the default term in it, as opposed to being blank, which it is now. if someone tries to void the box, just put the default term back in there, but have it look subdued. (subdued would mean = default, while any user entry would not be subdued, unless the user entry also = the default). *** so this bug is about not having the box blank, and fixing the description for if the box isn't blank. i would add that changing the default term from "Various Artists" to ANYTHING ELSE would solve bug 9523 but that isn't the purpose of this bug. this bug is strictly about how the VA display option is implemented.
ps. if you want the user to have the ability to remove this special category from display altogether, perhaps you could allow them to void it...? but that would be fundamentally different behavior then what we see today, since the VA category always shows, right?? (unless... in the possible exception of if SBS finds nothing to classify as a comp??) anyway, probably outside the scope of this bug, but i thought i'd mention it.
Phil Meyer says: "Beware that the SBS setting to configure the name used for representing compilation albums (by default Various Artists), is used for two purposes (wasn't a good idea). Not only is it the label to use for browsing a list of compilation albums, the scanner will assign this name as the album artist when it decides an album is a compilation." so aside from the previous two comments i made, i wanted to note this as well, since i agree with Phil a distinction should be made between these two functions, (meaning, it should be split into two settings).
Commit 96c2875743a8ab6054d1efe3349b04c3ba473002 - show default value as placeholder instead of empty value.