Issue #156 (Smart Playlist editor)

  Thread Rating:
  • 0 Votes - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Post Reply
MacGyver Offline
Fan
Posts: 475
Joined: Jan 2007
Reputation: 2
Location: Weilerbach Germany
Post: #1
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:
[Image: attachment.php?aid=288]


When focus is on the left:
[Image: attachment.php?aid=289]
Attached File(s) Thumbnail(s)
       

Using a 2820FYKH0 Intel NUC, a Revo 1600, and two Foxconn NT-330i - All running OpenELEC. :)
(This post was last modified: 2014-03-13 20:52 by MacGyver.)
find quote
jingai Offline
Skilled Skinner
Posts: 1,071
Joined: Mar 2010
Reputation: 12
Post: #2
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.
find quote
MacGyver Offline
Fan
Posts: 475
Joined: Jan 2007
Reputation: 2
Location: Weilerbach Germany
Post: #3
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/...df8915b34b

Using a 2820FYKH0 Intel NUC, a Revo 1600, and two Foxconn NT-330i - All running OpenELEC. :)
find quote
jingai Offline
Skilled Skinner
Posts: 1,071
Joined: Mar 2010
Reputation: 12
Post: #4
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.
find quote
MacGyver Offline
Fan
Posts: 475
Joined: Jan 2007
Reputation: 2
Location: Weilerbach Germany
Post: #5
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.

Using a 2820FYKH0 Intel NUC, a Revo 1600, and two Foxconn NT-330i - All running OpenELEC. :)
find quote
jingai Offline
Skilled Skinner
Posts: 1,071
Joined: Mar 2010
Reputation: 12
Post: #6
Just removing the line and arrow is probably fine, IMHO.
find quote
MacGyver Offline
Fan
Posts: 475
Joined: Jan 2007
Reputation: 2
Location: Weilerbach Germany
Smile  RE: Issue #156 (Smart Playlist editor)
Post: #7
Done.
https://github.com/MacGyverr/Metropolis/...8caea12703

[Image: attachment.php?aid=290]

[Image: attachment.php?aid=291]

[Image: attachment.php?aid=292]
Attached File(s) Thumbnail(s)
           

Using a 2820FYKH0 Intel NUC, a Revo 1600, and two Foxconn NT-330i - All running OpenELEC. :)
(This post was last modified: 2014-03-14 01:23 by MacGyver.)
find quote