Announce: Material Skin

[edwin2006]

Now playing looking awesome on desktop device!! Thanks for creating this view.

But... Not so much when using a mobile device. Would the desktop now-playing layout also be possible on a mobile device?

Edit. Also changing windows size on desktop give some layout issues like overlapping song info on art. Fullscreen landscape desktop all is perfect.

[fphredd]

Might be worth a Poll - what is the first menu item you use?

My GoTo's, in order:

1. Album Artists (I use ArtistSort tag to go by last names)
2. New Music
3. Compilations (Soon to be replaced by Spotty)

Hope this helps the decision making. Either way, thanks for Material Skin!

[SlimChances]

1. New Albums (Doesn't function now - a component of Custom Browse)
2. Albums (Doesn't function - a component of Custom Browse)
3. Music Folder

reason I use Custom Browse for these choices is that with MultiLibrary (Erland's plugin) I am able to select either from my FLAC or MP3 libraries and can keep them separate

[netchord]

Qobuz Search

I'm unable to search Qobuz from the material skin. i'm using a Synology DS216J NAS, and LMS 7.9.2. i've tried Chrome and Safari and neither works. click the search option in the Qobuz plugin and nothing happens. reverting to the default skin works fine.

it doesn't work on Mobile either.

[RobbH]

Might be worth a Poll - what is the first menu item you use?

1. Almost always: Music Folder
2. Distant second: Artists
3. Much more distant third: Albums
4. (default UI only): Advanced Search

[mherger]

Announce: Material Skin

> I'm unable to search Qobuz from the material skin.

I can confirm this. The click() function would simply return if an item
was of type search. Search items must be treated somewhere else?

--

Michael

[Ron F.]

I'm unable to search Qobuz from the material skin. i'm using a Synology DS216J NAS, and LMS 7.9.2. i've tried Chrome and Safari and neither works. click the search option in the Qobuz plugin and nothing happens. reverting to the default skin works fine.

it doesn't work on Mobile either.

I can verify this. The "Search" button for Qobuz no longer works - nothing happens. I had to flush all cookies and browsing data first however, and refresh the tab running the Material skin - and then it no longer worked using the 0.9.1 release.

Aside from that, question: is it possible in a future release for the Qobuz Search option to work more like the Search option in Spotty, where I can see a some kind of list of prior text searches? Is this something that would have to be added to the Qobuz plugin itself?

Anyway, thank you Craig for all your hard work - it certainly shows, what a fabulous job you have done.

[mherger]

Announce: Material Skin

> Aside from that, question: is it possible in a future release for the
> Qobuz Search option to work more like the Search option in Spotty, where
> I can see a some kind of list of prior text searches? Is this something
> that would have to be added to the Qobuz plugin itself?

Yes, that's a feature of the individual services. Not available everywhere.

--

Michael

[Ron F.]

> Aside from that, question: is it possible in a future release for the
> Qobuz Search option to work more like the Search option in Spotty, where
> I can see a some kind of list of prior text searches? Is this something
> that would have to be added to the Qobuz plugin itself?

Yes, that's a feature of the individual services. Not available everywhere.

--

Michael

Thank you for the info Michael.

-Ron

[cpd73]

> I'm unable to search Qobuz from the material skin.

I can confirm this. The click() function would simply return if an item
was of type search. Search items must be treated somewhere else?

Yeah, search is handled differently - as its not a clickable area, it should be a text field, and search performed upon pressing return.

I assume Qobuz search worked with 0.9.0? Is the search in the 1st level of Qobuz items (i.e. in the list of items after entering the Qobuz app)? Does this list allow toggling from grid to list? i.e. is the "Toggle view" button in the sub-toolbar, or in the main 3-dotted menu on the right in the upper toolbar?

For 0.9.1 I changed some code so that the view recycler is used more often - as this is much quicker when changing from list to grid, and back. I'm thinking this change might be the cause of the issue.

[mherger]

Announce: Material Skin

> Yeah, search is handled differently - as its not a clickable area, it
> should be a text field, and search performed upon pressing return.

Oh, that might explain it: Qobuz is another of those implementations
which uses icons, resulting in a grid view. Is search covered for grids?

> I assume Qobuz search worked with 0.9.0? Is the search in the 1st level
> of Qobuz items (i.e. in the list of items after entering the Qobuz app)?

Yes, 1st level.

> Does this list allow toggling from grid to list? i.e. is the "Toggle
> view" button in the sub-toolbar, or in the main 3-dotted menu on the
> right in the upper toolbar?

Yes, it can be toggled. But Search still wouldn't work.

--

Michael

[cpd73]

> Does this list allow toggling from grid to list? i.e. is the "Toggle
> view" button in the sub-toolbar, or in the main 3-dotted menu on the
> right in the upper toolbar?

Yes, it can be toggled. But Search still wouldn't work.

No, then it won't work. To speed up the recyler, I removed as many "if" clauses as possible - as I mainly want the recylcer for lists of the same type. So, when this is used, the search part is not wired in. I think I might see the error. Can you edit "browse-resp.js" and move line 341 ("types.add(i.type);") to line 340 - (i.e so that it is now before the closing bracket, not after).

[cpd73]

..obviously the above edits would be for the non-minified git checkout version, not the released code...

[mherger]

Announce: Material Skin

> No, then it won't work. To speed up the recyler, I removed as many "if"
> clauses as possible - as I mainly want the recylcer for lists of the
> same type. So, when this is used, the search part is not wired in. I
> think I might see the error. Can you edit "browse-resp.js" and move line
> 341 ("types.add(i.type);") to line 340 - (i.e so that it is now before
> the closing bracket, not after).

Yes, that would work. I'd no longer get the grid view, but a text input
for search instead.

--

Michael

[cpd73]

> No, then it won't work. To speed up the recyler, I removed as many "if"
> clauses as possible - as I mainly want the recylcer for lists of the
> same type. So, when this is used, the search part is not wired in. I
> think I might see the error. Can you edit "browse-resp.js" and move line
> 341 ("types.add(i.type);") to line 340 - (i.e so that it is now before
> the closing bracket, not after).

Yes, that would work. I'd no longer get the grid view, but a text input
for search instead.

Great! I'll commit this for the next release, thanks for the confirmation. Silly typo...