+- Kodi Community Forum (https://forum.kodi.tv)
+-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33)
+--- Forum: Skins Support (https://forum.kodi.tv/forumdisplay.php?fid=67)
+---- Forum: Metropolis (https://forum.kodi.tv/forumdisplay.php?fid=162)
+---- Thread: Issue #156 (Smart Playlist editor) (/showthread.php?tid=189058)
Issue #156 (Smart Playlist editor) - MacGyver - 2014-03-13
As per https://github.com/stoli/Metropolis/issues/156
I made some changes to the playlist editor in an attempt to make it less confusing. As you move down the list, it changes as the pictures below will show.
The request was really subjective so it's hard to tell what they want. Is this on the right track? (I'm not commiting them to my branch until I know it's what you're looking for.
When focus is on the right:
When focus is on the left:
RE: Issue #156 (Smart Playlist editor) - jingai - 2014-03-13
To be honest, it was so long ago, and the original request even then was a bit vague; however, what you've done does seem to improve it.
RE: Issue #156 (Smart Playlist editor) - MacGyver - 2014-03-13
Is the use of the bar and arrow in line with the look of Metropolis? I have no artistic ability so I have no idea. It just seemed like a practical cue, but if it's out of place I can try to think of something else.
I'll push it to my branch, and modify it as you need.
Here's the commit.
https://github.com/MacGyverr/Metropolis/commit/81694d43a9f8077b4111ccb05ef64cdf8915b34b
RE: Issue #156 (Smart Playlist editor) - jingai - 2014-03-13
No, I'd say it's not really in-line with the rest of it. Whether or not its better I suppose could be a matter of debate, but I'm not going to make that change globally at this time.
RE: Issue #156 (Smart Playlist editor) - MacGyver - 2014-03-13
I can easily remove the image code, and modify it to either just keep hiding them like it current does, or try to dim-out the inaccessible objects.
RE: Issue #156 (Smart Playlist editor) - jingai - 2014-03-13
Just removing the line and arrow is probably fine, IMHO.
RE: Issue #156 (Smart Playlist editor) - MacGyver - 2014-03-13
Done.
https://github.com/MacGyverr/Metropolis/commit/fe56139da390a86025b785ddde7b4b8caea12703