Daniel Foré [12:19 PM]
@carl: hm you're right. wtf.

Daniel Foré [12:19 PM]12:19
I think @harvkitty has smart playlist editor mocks

Harvey Cabaguio [12:23 PM]
yes
Harvey Cabaguio
[new playlist.png]

Harvey Cabaguio [12:24 PM]
I need to sit down and make these blueprints

Daniel Foré [12:24 PM]
Yeah

Daniel Foré [12:24 PM]
that's sexy like a chocolate strawberry

Carl [12:29 PM]
So I guess a bug report should be filled. Also sort order should be added.

Harvey Cabaguio [12:53 PM]
@carl: as part of the rules? how would it work?

Harvey Cabaguio [1:07 PM]
@danrabbit: does this make sense https://blueprints.launchpad.net/noise/+spec/combine-new-playlist-and-smartplaylist-ui
Currently, the method for creating a playlist and smart playlist in Noise is a bit different. We need to combine them into one simple UI to make things simpler. TODO: Smart Playlist * Use a popover instead of a separate window for making a new playlist * Have the smart playlist icon in the entry to show that its a smart playlist * Allow rules to be removed * Make the playlist size limit a part of the rules * When all the rules are removed, revert back to a normal playlist (show normal playl...

Harvey Cabaguio [1:07 PM]
I'm gonna file the todo as bugs (edited)

Sergey Davidoff [1:07 PM]
the "remove rule" button should probably turn red on hover, and _ideally_ there should be some kind of undo mechanism too. (edited)

Harvey Cabaguio [1:09 PM]
I don't think an undo would be necessary

Sergey Davidoff [1:09 PM]
I have to admit it does feel like over-engineering

Carl [1:16 PM]
@harvkitty when you create a smart playlist that returns the last played you might want to have it sorted by last played and rating.

Carl [1:18 PM]
@harvkitty I agree with shnatsel the fusion seems over engineered.

Harvey Cabaguio [1:18 PM]
is it?

Sergey Davidoff [1:19 PM]
What fusion? I was referring to the "undo" thing in there. That's probably over-engineering. I have no opinion on smart playlists because I don't use them. (edited)

Harvey Cabaguio [1:21 PM]
@carl: I was basically like, the only thing separating smart playlists from normal playlists is that smart ones have rules. no rules > normal playlist

Carl [1:21 PM]
Fusing limit with rules makes things more complicated as the ui mixes things that are not the same: clauses that apply on field and a limit that apply to a smart playlist.

Carl [1:22 PM]
It also crowd the rule list that is already long as it is.

Sergey Davidoff [1:23 PM]
Do we even need a track number limit? I can't think of any use cases for it, and this Noise is fast enough to handle almost any track lists that fit on a modern HDD

Carl [1:23 PM]
And the thing that you can drag and drop tracks on a playlist.

Carl [1:24 PM]
Noise 0.4 is fast/smart enough. Noise 0.3 needed the artificial limit. And badly so.

Harvey Cabaguio [1:24 PM]
so we can remove the limit?

Carl [1:24 PM]
In a future release yes.

Sergey Davidoff [1:25 PM]
you mean in your current experimental branch, or that's a totally future work item? (edited)

Carl [1:25 PM]
Also you can't turn a static playlist into a smart playlist: what happens to the tracks that have been added to the static playlist?

Harvey Cabaguio [1:26 PM]
this is only when you're making a playlist

Carl [1:26 PM]
It works right now if my current branch without the limit: I tested it with huge smart playlists.

Harvey Cabaguio [1:27 PM]
if you're editing the rules of a smart playlist you could make them unable to remove the last rule

Sergey Davidoff [1:27 PM]
Regular and smart playlists are separate entities still, you just have a single dialog to create any of those.

Sergey Davidoff [1:27 PM]
They should even have different icons (in 0.3 they do)

Carl [1:32 PM]
To be fair the actual playlist / smart playlist creation process is not complex (for all its woes). While I think that the dialog must be redone (and the blueprint is totally useful) it should be reworded as its goal is not to make things simpler. (edited)

Carl [1:32 PM]
(Just a question of being accurate)
Harvey Cabaguio
[1:32 PM]
[new playlist.png]

Harvey Cabaguio [1:33 PM]
isn't this simpler?

Sergey Davidoff [1:39 PM]
now that I think of it, maybe not

Sergey Davidoff [1:40 PM]
I like the fact that if I need a simple dumb playlist it just takes me directly there and the title editing happens in the sidebar. Zero superfluous entities!

Daniel Foré [1:41 PM]
Well it's the same number of clicks at the moment

Daniel Foré [1:41 PM]
you hit add and then decide if it's a smart playlist or not

Harvey Cabaguio [1:42 PM]
and its consistent with everything else that has that +- bar
Sergey Davidoff
[1:42 PM]
[sergey's mockup]

Sergey Davidoff [1:43 PM]
To be honest that +/- bar seems like a terrible UI element to me. It's so far away from the actual playlists that I just don't get the connection between them.

Sergey Davidoff [1:43 PM]
I'd rather show a _Create new playlist..._ item at the bottom of the list of playlists (edited)

Corentin Noël [1:44 PM]
The problem is that it's in a wide status bar

Sergey Davidoff [1:45 PM]
Yeah, in Noise it's doubly confusing because it's in the status bar. But it's a terrible UI element in general; the controls are too detached from the actual content.

Corentin Noël [1:45 PM]
Well, The element to add elements was really weird in the Application plug

Sergey Davidoff [1:46 PM]
Just type the name in _Italics..._ and it clicks :wink:

Corentin Noël [1:46 PM]
talk about Italic to dan… (edited)

Sergey Davidoff [1:47 PM]
What if we use a _Create new playlist..._ item in the list of playlists instead of that + in the statusbar and modify the "No songs" notice to include a button to create a smart playlist?

Sergey Davidoff [1:47 PM]
danrabbit: :point_up:

Corentin Noël [1:47 PM]
I would say, how often do you create playlists

Carl [1:48 PM]
I would say often.

Sergey Davidoff [1:48 PM]
I never do, I just don't see the controls to do that.

Corentin Noël [1:48 PM]
really ?

Carl [1:49 PM]
But noise may have trained users not to as they are deleted in some cases. (edited)

Carl [1:50 PM]
Yes of course. A playlist for particular mood. Or just to keep all heterogeneous tracks sitted in the same folder together!

Corentin Noël [1:50 PM]
Are you really creating playlists daily ?

Corentin Noël [1:51 PM]
Then you're working at Spotify or so :stuck_out_tongue:

Carl [1:51 PM]
I use them daily! I tweak them from time to time.

Sergey Davidoff [1:51 PM]
I don't create playlists daily, but when I do, I want the program to get the hell out of my way and just let me do it quickly and efficiently.

Carl [1:52 PM]
@shnatsel and I don't want to loose a playlist ever again. That's data loss.

Sergey Davidoff [1:52 PM]
I feel ya.

Carl [1:54 PM]
My playlists and the media tags are my content/document. Each time noise looses them it's data loss. (edited)

Carl [1:54 PM]
So not saving the media tag rating (and skip/play count) in the file is data loss.

Carl [1:55 PM]
Deleting playlists because I change my music folder is data loss.

Carl [1:55 PM]
Asking me to rm my db for any reason is data loss.

Daniel Foré [1:56 PM]
slackbot, what do we say about italic

Slack Bot [1:56 PM]
https://www.youtube.com/watch?v=umDr0mPuyQc
YouTube
Retro Pep
NO GOD! PLEASE NO!!! NOOOOOOOOOO


Corentin Noël [1:57 PM]
carl: I've nothing about keeping your playlists alive :stuck_out_tongue:
I just say we don't need a fake item in the menu to add more (edited)

Daniel Foré [1:58 PM]
^

Carl [1:58 PM]
I was not replying to the menu thing. Either way it boils down to the same thing.

Carl [1:58 PM]
I was just talking about a bigger concern of mine.

Carl [2:00 PM]
The option is a bit tiny as shnatsel said. But I would rather go with what is consistent with the os. So the + - thing. I like the right click delete though (that is currently implemented) (edited)

Sergey Davidoff [2:09 PM]
danrabbit: how about creating a regular playlist right away and showing that instructions screen (screenshot above) with an "Automatically fill playlist" button?
So that a playlist is created right away and then you can choose if you want to populate it manually or define the auto-fill rules. Plus this rids us of an extra entity in the form of that popover dialog.

Corentin Noël [2:12 PM]
or display a button in the empty view allowing to populate it

Sergey Davidoff [2:12 PM]
tintou: that's exactly what I meant

Corentin Noël [2:12 PM]
shnatsel: so it's a good idea :smile:

Sergey Davidoff [2:15 PM]
I still suck at explaining don't I

Corentin Noël [2:15 PM]
Maybe did I failed to understand you too :simple_smile: (edited)

Corentin Noël [2:17 PM]
but the only thing I can tell you is Da, Niet (edited)

Daniel Foré [2:18 PM]
@shnatsel: that's an interesting idea, however

Daniel Foré [2:18 PM]
what about editing?

Daniel Foré [2:18 PM]
now we must have two interfaces

Sergey Davidoff [2:19 PM]
We could just move Harvey's popover to that button in the empty view so we still have one UI. But I think there's a better way to do that.

Sergey Davidoff [2:20 PM]
let me do a quick mock
Sergey Davidoff
[2:25 PM]
Quick mock

 93KB PNG in #noise  • Add comment • Open original

Sergey Davidoff [2:26 PM]
danrabbit: :point_up: This has one huge advantage: live preview of your filters, both when creating and when editing

Daniel Foré [2:28 PM]
how is that somehow better than using the popover?

Sergey Davidoff [2:28 PM]
carl: is that a reasonable thing from the technical standpoint, or will require too much work?

Daniel Foré [2:29 PM]
it's just bigger

Sergey Davidoff [2:30 PM]
danrabbit: no wonky focus issues, won't collapse until you're done or you've cancelled it (forgot the Finish and Cancel buttons in the mock!) and has a clear connection to the track list

Carl [2:41 PM]
Well it will require work. But there is no performance issue here. I would suggest to move the criteria above as the media are a result (so the bottom is more fitting)

Carl [2:41 PM]
That being said I probably won't be the one implementing the new playlist creation dialog.

Carl [2:42 PM]
(I have had enough of noise for many lifetimes)

Corentin Noël [2:42 PM]
carl: oh !

Sergey Davidoff [2:43 PM]
Then the entire thing should be postponed post-Freya and it's pointless to discuss it now

Carl [2:43 PM]
Well all this interesting chat must added to the blueprint. So it won't be lost in slack guts.