2017-08-05, 02:52
(2017-08-04, 00:38)DaveBlake Wrote: Good to have someone work through this, thanks Scott.
[My response edited for clarity by light of day]
1) What "Set for all albums shown" means:
"Change info provider" is available from any node with content = "albums" e.g. even a smart playlist showing just albums with year = 1960. This option is to set the scraper addon settings for just those (possibly filtered) albums. I guess the confusion is does it mean on the page, or all pages?
Setting the "Set default info provider" (3rd button) changes the default addon setting, and removes all previously set album settings, not just those shown (possibly filtered) on the node.
As you found, from the albums library node (which is unfiltered) "Set for all albums shown" causes all albums to be given an info setting, so is much the same result as "Set default info provider" for those albums, but does not change the addon settings. Hence a newly added album would be still have addon settings set to the unchanged default values.
a) Is that confusing? Should the "Set for all albums shown" button be hidden when all albums are showing on the node?
I would suggest "Set for all albums/artists in current view" though maybe that's too long. Now that I see the intent, it works very well.
And both buttons are different from changing the default provider from the settings menu (in the two separate places - which addon to use, and the settings). That just changes the addon and any defaut settings, but does not change the library. It does not remove previously set album settings, nor offer refresh, but what is offered on the context menu does change the library contents hence the comfirmation dialogs.
So
b) Is that reasonable?
c) How can it best be made clear? As you point out the number of characters visible on the label is limited, so what else could these context menu buttons be called? Or should it be using some other UI approach?
It is really a skinner issue, but having to format a long label makes a complication for the skinner (and there are unofficial Estuary mods using font with larger point size which compounds the issue. I was able to get a PR approved that adds a label for fontset in use, so a skinner could reformat his/her labels based on font size.
I'm not sure I see exactly what we do now with info provider setings. When are they stored in the addon settings, and when in the library? I was testing with Uni Album Scraper and it didn't seem right, but I'm not sure.
2) We need to keep the settings for those that want the "fetch online info on update" when adding the first music source, as need somewhere to set what scraper addon will be used. Initially when no source the albums nodes will all be empty, so no context menu route available.
d) But maybe the change from settings should also clear and refresh any existsing albums or artists? Although generally settings changes don't get applied to the library retrospectively when they are changed
I added some comments in the quoted text, but need to do some more testing.
scott s.
.