feat: separate 'clone' and 'create' when creating a new playist

[dhanuarf]

This PR separate clone playlist and create playlist into separate view via spinner.

Having 2 options in the same UI while the user only able to proceed with just one of them is kind of bad UX, I think?

Before changes:

After changes:

[Bnyro]

Perhaps the spinner option to create a new playlist should be renamed to something like New private playlist? That probably makes it more clear, "Create playlist" is already in the title and thus we currently have the same text twice.

[dhanuarf]

Perhaps the spinner option to create a new playlist should be renamed to something like New private playlist? That probably makes it more clear, "Create playlist" is already in the title and thus we currently have the same text twice.

Yeah I was trying to avoid to create or modify any strings, so I just used whats available.

[dhanuarf]

c86f442:

[Bnyro]

Suggested change

	binding.createPlaylistSpinner.items = listOf<String>(
	getString(R.string.clonePlaylist),
	getString(R.string.createNewPlaylist))
	binding.createPlaylistSpinner.items = listOf<String>(
	getString(R.string.createNewPlaylist),
	getString(R.string.clonePlaylist)
	)

The default value should be the first item, not the second one. It feels a bit confusing otherwise.

[Bnyro]

Thank you!